Skip to content

Conversation

@mbutrovich
Copy link
Contributor

DataFusion is preparing for their 51.0.0 release, so this is a test PR to see how CI goes with main on a recent commit that likely reflects what DF 51 will be. Marking as draft since we wouldn't want this merged until crates are available and the community discusses the changes.

Which issue does this PR close?

  • N/A.

What changes are included in this PR?

  • Bump DF to 51
  • Bump Arrow/Parquet to 57
  • Page index tests are skipped right now because of API changes in Arrow-rs. I will have to revisit what we can do to fix these since they were relying on a pretty low-level API
  • Toolchain bump to support DF's 1.88. I need to understand if this would be within iceberg-rust's MSRV policy

Are these changes tested?

Creating this to run CI.

@alamb
Copy link

alamb commented Nov 8, 2025

i can probably help with the page index changes

datafusion = "50"
datafusion-cli = "50"
datafusion-sqllogictest = "50"
datafusion = { git = "https://github.com/apache/datafusion.git", rev = "f32984b2dbf9e5a193c20643ce624167295fbd61" }
Copy link
Contributor

Choose a reason for hiding this comment

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

just for my own curiosity, is it because latest stable release datafusion doesn't play with the other dependencies, that we have to use commit sha here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

branch-51 didn't exist yet, so I just picked whatever the current commit on main was at the time. I could run it again with DataFusion's branch-51, but not expecting different results.

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