-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update to arrow-7.0.0 #1523
Update to arrow-7.0.0 #1523
Conversation
wait for this pr to be merged. |
This PR is now ready for review / merge |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @alamb for doing this!
thanks, @alamb I can go on works depending on arrow-rs 7.0.0. |
Sorry it took so long @liukun4515 -- I hope this delay will be reduced by the proposal in apache/arrow-rs#1120 Thank you for all your contributions and your support |
totally agree with your proposal. |
Draft until arrow-7.0.0 is actually released.This includes code from #1283
Once arrow 7 is released (hopefully next week) I'll update this PR to use it and mark as ready to review. Until that time feel free to rebase any branches off this one if you need new features.This PR is now ready for review / merge
Rationale for this change
Update to get new code in arrow
What changes are included in this PR?
prost
dependency 0.8 --> 0.9 andtonic
from 0.5 --> 0.6Are there any user-facing changes?
requires new arrow