-
Notifications
You must be signed in to change notification settings - Fork 198
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
Bump DataFusion version #453
Conversation
Looks great - I went ahead and fixed the cargo.toml bug - this uncovered some necessary changes needed in the python crate. I believe the other failure is apache/datafusion#3953 |
This brings also some perf improvements to q7 and q19. q15 now runs too 🥳 |
Looks great. One more thing about the format of the datafusion.proto, is there any standard format so that future PRs will not be influenced by the format change? |
I also pushed changes to compile python and updated revs to include apache/datafusion#3978 |
The earlier failing test passes now too |
Thanks @Dandandan ! |
Filed #462 for this |
LGTM. I cannot approve because I created the PR. |
Which issue does this PR close?
N/A
Rationale for this change
I would like to test some of the bug fixes in DF and see if more of the benchmark queries now work.
What changes are included in this PR?
Use DF just before apache/datafusion#3907 because we are talking about rolling that back.
Are there any user-facing changes?
I don't know yet