-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
GH-34171: [Go][Compute] Implement "Unique" kernel #34172
Conversation
Benchmark runs are scheduled for baseline = 266f166 and contender = 90071cc. 90071cc 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. |
### Rationale for this change Implementing a kernel for computing the "unique" values in an arrow array, primarily for use in solving apache#33466. ### What changes are included in this PR? Adds a "unique" function to the compute list and helper convenience functions. ### Are these changes tested? Yes, unit tests are included. ### Are there any user-facing changes? Just the new available functions. * Closes: apache#34171 Authored-by: Matt Topol <zotthewizard@gmail.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
### Rationale for this change Implementing a kernel for computing the "unique" values in an arrow array, primarily for use in solving apache#33466. ### What changes are included in this PR? Adds a "unique" function to the compute list and helper convenience functions. ### Are these changes tested? Yes, unit tests are included. ### Are there any user-facing changes? Just the new available functions. * Closes: apache#34171 Authored-by: Matt Topol <zotthewizard@gmail.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Rationale for this change
Implementing a kernel for computing the "unique" values in an arrow array, primarily for use in solving #33466.
What changes are included in this PR?
Adds a "unique" function to the compute list and helper convenience functions.
Are these changes tested?
Yes, unit tests are included.
Are there any user-facing changes?
Just the new available functions.