-
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-15984: [C++] Change RecordBatchReader API to use Result<> #12678
Conversation
Can you take a look at the C/GLib CI failure here? https://github.com/apache/arrow/runs/5645554934?check_suite_focus=true Also, usage in PyArrow should probably be migrated also, do you want to do it yourself? (otherwise, someone else can) |
7dc295c
to
cc9fb71
Compare
regarding Python and GLib bindings, I'll try doing these myself |
c6c9831
to
a75e151
Compare
regualar tests no longer cover this method
ASSERT_OK_NO_THROW use was unnecessary in the first place
a75e151
to
b69e257
Compare
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 @zagto
Benchmark runs are scheduled for baseline = 7711182 and contender = 623a15e. 623a15e is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.