-
Notifications
You must be signed in to change notification settings - Fork 803
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
Implement filter
kernel for byte view arrays.
#5624
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.
Thank you for this contribution @RinChanNOWWW 🙏 This looks really nice
The only thing I think we need to do to merge this PR is to run the filter_kernels benchmarks to show that this doesn't regress existing performance
It would be a nice bonus to add a new benchmark for filtering of BinaryView and StringView array
dd74d49
to
9a6fd81
Compare
BenchmarkTest MachineMacBook M1 Pro (10 Cores, 32G RAM) Bench ResultsBefore this PR
This PR
|
Do the benchmark results meet expectation? @alamb |
Thanks @RinChanNOWWW -- much appreciated THanks for the merge @tustvold |
Which issue does this PR close?
Closes #5510.
Rationale for this change
Necessary feature.
What changes are included in this PR?
filter
kernel forStringViewArray
andByteViewArray
.filter_kernels
.Are there any user-facing changes?
Yes.
Deprecate
ArrowPrimitiveType::get_byte_width
. Move this method toArrowNativeType::get_byte_width
.When using
ArrowPrimitiveType
, we can get the byte width byArrowPrimitiveType::Native::get_byte_width
.