Skip to content

Backport #3924 to pyiceberg-0.12.x - #4016

Open
illes wants to merge 1 commit into
apache:pyiceberg-0.12.xfrom
illes:backport-3924-pyiceberg-0.12.x
Open

illes wants to merge 1 commit into
apache:pyiceberg-0.12.xfrom
illes:backport-3924-pyiceberg-0.12.x

Conversation

@illes

@illes illes commented Sep 24, 2026

Copy link
Copy Markdown

Rationale for this change

Cherry-pick of #3924 (db5bd045) for a 0.12.1, as discussed on that PR. Hive 4.0.1 removed get_table and get_table_objects_by_name, so the 0.12.0 HiveCatalog fails against Hive 4.0.1+ metastores with Invalid method name.

Are these changes tested?

Same tests as #3924. tests/catalog/test_hive.py passes on this branch.

Are there any user-facing changes?

HiveCatalog uses get_table_req / get_table_objects_by_name_req unless hive.hive2-compatible is set, as on main. hive.hive2-compatible is only needed for metastores older than Hive 2.3.0 (July 2017), the first release with the _req methods.

* Update hive docker to 4.2.1

* Use newest Hive docker image

* Add back MAVEN_MIRROR

(cherry picked from commit db5bd04)
@illes

illes commented Sep 24, 2026

Copy link
Copy Markdown
Author

@rambleraptor, the integration jobs fail there because Docker Hub now denies the minio/mc image pull. main replaced MinIO with RustFS in #3928, and that also cherry-picks cleanly onto the branch. Should I add it to #4016, or would you rather keep it as a separate backport?

@rambleraptor

Copy link
Copy Markdown
Collaborator

@illes let's do two PRs if you don't mind. I'd like to keep the commit history relatively sane.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants