Skip to content

chore(deps): update datafusion to 52.4.0 [iceberg]#3769

Merged
andygrove merged 3 commits intoapache:mainfrom
andygrove:df-52.4.0
Mar 23, 2026
Merged

chore(deps): update datafusion to 52.4.0 [iceberg]#3769
andygrove merged 3 commits intoapache:mainfrom
andygrove:df-52.4.0

Conversation

@andygrove
Copy link
Member

Update the DataFusion dependency from 52.3.0 to 52.4.0. No API changes were required.

Copy link
Contributor

@mbutrovich mbutrovich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @andygrove!

@mbutrovich mbutrovich changed the title chore(deps): update datafusion to 52.4.0 chore(deps): update datafusion to 52.4.0 [iceberg] Mar 23, 2026
@mbutrovich
Copy link
Contributor

mbutrovich commented Mar 23, 2026

Added [iceberg] to the title and merged in latest main to kick off CI again. We should probably get in the habit (as long as we don't have them running by default) to tag [iceberg] on any DataFusion bumps. I'm not as concerned about a minor version, but still.

@mbutrovich
Copy link
Contributor

mbutrovich commented Mar 23, 2026

Looks like we need new iceberg-rust diffs after #3739. CometIcebergNativeScan does not need Iceberg Java changes, but I think the diffs were still based on the Iceberg Java integration to reduce the differences. We could either 1) generate new diffs that only have the test changes we need to test that codepath 2) peel back the changes for Iceberg Java integration from the existing diffs to get it compiling again.

At this point option 1 might be less work.

Copy link
Contributor

@comphead comphead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @andygrove pending CI

@andygrove andygrove merged commit dc7d54c into apache:main Mar 23, 2026
152 checks passed
@andygrove andygrove deleted the df-52.4.0 branch March 23, 2026 23:49
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.

3 participants