-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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-5594: [C++] add UnionArrays support to Take/Filter kernels #4866
ARROW-5594: [C++] add UnionArrays support to Take/Filter kernels #4866
Conversation
ed180da
to
85fe336
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.
Thank you @bkietz ! Looks good on the principle, just a couple comments and questions.
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. Will merge if green.
Codecov Report
@@ Coverage Diff @@
## master #4866 +/- ##
==========================================
+ Coverage 87.46% 89.1% +1.64%
==========================================
Files 994 717 -277
Lines 140389 100548 -39841
Branches 1418 0 -1418
==========================================
- Hits 122788 89591 -33197
+ Misses 17239 10957 -6282
+ Partials 362 0 -362
Continue to review full report at Codecov.
|
Take and Filter kernels should now be able to operate on arrays of any type. Closes apache#4866 from bkietz/5594-add-support-for-UnionArrays-to-Take-and- and squashes the following commits: 2e2e589 <Benjamin Kietzman> correct union density test 1c0b29f <Benjamin Kietzman> address review comments 1e07cfb <Benjamin Kietzman> add tests for dense union array density 3df7bf3 <Benjamin Kietzman> add Take/Filter(UnionArrays) Authored-by: Benjamin Kietzman <bengilgit@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Take and Filter kernels should now be able to operate on arrays of any type.