Skip to content

chore: Upgrade datafusion to version 49 #1200

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nuno-faria
Copy link
Contributor

Which issue does this PR close?

N/A.

Rationale for this change

Upgrades to the latest version of datafusion and related crates.
Marked as a draft for now while we wait for 49.0.1, which fixes a bug that is causing some tests to fail: apache/datafusion#17036.

What changes are included in this PR?

  • Upgraded datafusion and related crates to version 49.
  • Migrated code per the upgrade guide.

Are there any user-facing changes?

The Python API remains the same. One thing to notice is that md5 type is now string_view instead of string.

@timsaucer
Copy link
Contributor

Thank you for taking this on! I was just thinking today it was time to kick off the 49 release. I suspect that failing test is due to the upstream changes.

@nuno-faria
Copy link
Contributor Author

I suspect that failing test is due to the upstream changes.

Yes, the specific bug causing the errors is being tracked here: apache/datafusion#17011.

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