Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Nov 17, 2025

Rationale for this change

register_table_provider has been deprecated: https://datafusion.apache.org/python/autoapi/datafusion/context/index.html#datafusion.context.SessionContext.register_table_provider

image

Are these changes tested?

Are there any user-facing changes?

@kevinjqliu
Copy link
Contributor

ah this is deprecated in datafusion 50.x

The next release of pyiceberg-core (0.8) will have datafusion 50.x as a dependency
https://github.com/apache/iceberg-rust/blob/e519c3aed12ec762b360e7ee9b53530b6214015e/Cargo.toml#L65

We can then upgrade pyiceberg's dependency to datafusion 50.x and deprecate this function.

kevinjqliu added a commit to kevinjqliu/iceberg-python that referenced this pull request Dec 23, 2025
Merge branch 'fd-datafusion' into kevinjqliu/iceberg-rust-rc
@kevinjqliu kevinjqliu added this to the PyIceberg 0.11.0 milestone Jan 20, 2026
Fokko added a commit that referenced this pull request Jan 20, 2026
<!--
Thanks for opening a pull request!
-->

<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->

# Rationale for this change
Update pyiceberg-core to the latest release, 0.8.0
Also update datafusion to match the version iceberg-rust uses, v51
https://github.com/apache/iceberg-rust/blob/v0.8.0/Cargo.toml#L65

Pulled in #2749 as well. Otherwise CI will fail.
`register_table_provider` has been deprecated:
https://datafusion.apache.org/python/autoapi/datafusion/context/index.html#datafusion.context.SessionContext.register_table_provider

## Are these changes tested?

## Are there any user-facing changes?

<!-- In the case of user-facing changes, please add the changelog label.
-->

---------

Co-authored-by: Fokko Driesprong <[email protected]>
@Fokko Fokko closed this Jan 20, 2026
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