-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-15776: [Python] Expose IpcReadOptions #12800
Conversation
|
…cordBatchFileReader docstrings
@jorisvandenbossche As discussed I think this can start review process. I was going to assign you as reviewer but I don't have permission. |
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.
This is a very nice PR, thanks!
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.
LGTM overall. (We should file a follow-up to expose this in Flight.)
…fault C++ value for included_fields instead of empty list
…e as part of open_stream and open_file directly
Actually, we should also add it to the docs here: arrow/docs/source/python/api/ipc.rst Line 41 in b829943
|
…uded_fields.clear as requested on code review
|
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.
Thanks for doing this @raulcd! Here are some comments.
Co-authored-by: Antoine Pitrou <pitrou@free.fr>
…_fields and use expected values for test
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.
Thanks! Two nits, otherwise LGTM.
Co-authored-by: Antoine Pitrou <pitrou@free.fr>
…s as requested on review
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.
+1, thank you @raulcd
Benchmark runs are scheduled for baseline = 6c10a38 and contender = 2a2c087. 2a2c087 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
['Python', 'R'] benchmarks have high level of regressions. |
This PR intends to expose IpcReadOptions to pyarrow.