Skip to content
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

Upgrade to datafusion 39 and arrow 52 #14

Closed
wants to merge 4 commits into from

Conversation

sergiimk
Copy link

Initial attempt to upgrade to datafusion v39 and latest jiter - in case you find this helpful.

Note the proposed change of dependency pinning to a specific major release, related to #12.

Also note that one test is failing - I could not yet figure out what am I doing wrong with the new way to initialize UnionArray.

@davidhewitt
Copy link
Collaborator

Oops looks like we raced a bit on preparing this one @sergiimk - I also opened #15 yesterday!

I agree with pinning to a specific major release, I did the same in my PR👍

@davidhewitt
Copy link
Collaborator

Superseded by #15, thanks for the contribution 🙏

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