-
Notifications
You must be signed in to change notification settings - Fork 4
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 Arrow to arrow-one-mmap-file-on-b228489 #1
Conversation
@guoyuhong Thanks for the patch! Now that ray-project/ray#3898 is merged, we can move forward with this. Before we can merge it, however we first need to fix the problem https://issues.apache.org/jira/browse/ARROW-4491 that happens with newer versions of arrow. If you have spare capacity to look into this let me know! I can only get to it at the end of the week I think, as I'm currently busy with ray-project/ray#4129, which I'm prioritizing to make sure we can exit the stage where we have to support both CMake and Bazel as soon as possible. |
@pcmoritz Thanks for the information. I'm not quite understand what the Arrow JIRA issue is about. If I have time in the next several days, I will take a look at that issue. |
@pcmoritz I have merged apache/arrow#3758 . Is that the correct solution? |
Thanks, yes that is working now! Great :) This PR is now integrated into the master of this repo too! |
Described here: ray-project/ray#4127 (comment)