-
Notifications
You must be signed in to change notification settings - Fork 921
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 Arrow 17 #16400
Upgrade to Arrow 17 #16400
Conversation
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.
I don't think we want to upgrade this, certainly not so quickly. We know that moving our Arrow dependency too quickly can cause pain for some of our users (CC @beckernick), and also we're targeting removing the libarrow dependency in 24.10 anyway.
Thanks @vyasr - I had the same concern. Let's try to reduce our Arrow dependencies first, and we can upgrade this later in the 24.10 release cycle if still needed. |
Sounds good 👍 |
@galipremsagar once #16640 merges, we should replace this PR with one that simply relaxes the pyarrow constraint. Would you be able to take point on testing the versions that we support? I expect that anything after pyarrow 13 should work for us (we need pyarrow 13 for apache/arrow#36162). @seberg when we add the earliest dependency testing to cudf (the cudf version of rapidsai/rmm#1613) can we also ensure that the appropriate pyarrow (and pandas) versions are being tested there? Thank you! |
Sure 👍 |
They already are part of gh-16570 (although you can double check the versions, and I guess this would modify them). |
Oh awesome I thought there was a cudf PR for this but couldn't find it. Thanks for pointing it out! Yes, we'd just want to update the versions there. |
Right this is the cuDF PR: #16570 Think Matthew is going to work on adding some skips and xfails as appropriate: #16570 (comment) Guessing once that is done it can be merged |
Replaced by #16681. |
Description
This PR upgrades to arrow-17 in cudf.
Checklist