Skip to content
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.

Sign up for GitHub

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

Jump to bottom

JNI for HISTOGRAM and MERGE_HISTOGRAM aggregations #14154

Merged
merged 109 commits into from
Sep 27, 2023
Merged

JNI for HISTOGRAM and MERGE_HISTOGRAM aggregations #14154

merged 109 commits into from
Sep 27, 2023

Conversation

ttnghia
Copy link
Contributor

@ttnghia ttnghia commented Sep 21, 2023 •
edited
Loading

This implements JNI for HISTOGRAM and MERGE_HISTOGRAM aggregations in both groupby and reduction.

Depends on:

Contributes to:

@ttnghia
Add COUNT_FREQUENCY and MERGE_FREQUENCY aggregations
e385fda
@ttnghia
Change the new aggregations to HISTOGRAM and MERGE_HISTOGRAM
e3df8d4
@ttnghia
Update copyright year
7bc7f91
@ttnghia
Implement interface for the new aggregations
0fd2000
@ttnghia
Add new files
1b04436
@ttnghia
Add skeleton APIs
1977d69
@ttnghia
Extract hash_reduce_by_row
6fa93fc
@ttnghia
Adopt hash_reduce_by_row in distinct_reduce
d11dd7f
@ttnghia
Merge branch 'branch-23.10' into percentile
b632570
@ttnghia
Rename struct and simplify code
e58f3e3
@ttnghia
Refactor hash_reduce_by_row
3cf1948
@ttnghia
Rewrite hash_reduce_by_row.cuh
8488646
@ttnghia
Rename and rewrite distinct_reduce.hpp
1994684
@ttnghia
Rewrite distinct.cu
5dcbac9
@ttnghia
Rewrite distinct_reduce.cu
6236fcc
@ttnghia
Rewrite hash_reduce_by_row.cuh
42a778f
@ttnghia
Minor changes
584ff8d
@ttnghia
Fix style
4a3d60d
@ttnghia
Merge branch 'refactor_hash_reduce' into percentile
4e74119 
# Conflicts:
#	cpp/src/reductions/hash_reduce_by_row.cuh
#	cpp/src/stream_compaction/distinct_reduce.cu
@ttnghia
Fix comment
34cb488
@ttnghia
Merge branch 'refactor_hash_reduce' into percentile
c863b53
@ttnghia
Move file
e73c07f
@ttnghia
Merge distinct_reduce.* into distinct.cu
40e8730
@ttnghia
Move file
95e4463
@ttnghia
Merge distinct_reduce.* into distinct.cu
723ae4c
@ttnghia
Merge branch 'refactor_hash_reduce' into percentile
921243e
@ttnghia
Revert "Merge distinct_reduce.* into distinct.cu"
8fb7a9e 
This reverts commit 723ae4c.
@ttnghia
Rename function
65427c8
@ttnghia
Merge branch 'refactor_hash_reduce' into percentile
bcc2db4
@ttnghia
Fix output type
0c0c7ac
@ttnghia ttnghia added feature request New feature or request 3 - Ready for Review Ready for review by team 0 - Blocked Cannot progress due to external reasons Java Affects Java cuDF API. Spark Functionality that helps Spark RAPIDS non-breaking Non-breaking change labels Sep 21, 2023
@ttnghia ttnghia requested a review from a team as a code owner September 21, 2023 16:57
@ttnghia ttnghia self-assigned this Sep 21, 2023
@ttnghia
Copy link
Contributor Author

ttnghia commented Sep 27, 2023

/merge

@rapids-bot rapids-bot bot merged commit 66ac962 into rapidsai:branch-23.10 Sep 27, 2023
57 checks passed
@ttnghia ttnghia deleted the jni_histogram branch October 16, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jlowe jlowe jlowe approved these changes

Assignees

@ttnghia ttnghia

Labels
0 - Blocked Cannot progress due to external reasons 3 - Ready for Review Ready for review by team feature request New feature or request Java Affects Java cuDF API. non-breaking Non-breaking change Spark Functionality that helps Spark RAPIDS
Projects
cuDF/Dask/Numba/UCX
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ttnghia @jlowe