-
Notifications
You must be signed in to change notification settings - Fork 915
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
JNI support for Collect Ops in Reduction #10427
JNI support for Collect Ops in Reduction #10427
Conversation
Signed-off-by: sperlingxx <[email protected]>
Codecov Report
@@ Coverage Diff @@
## branch-22.04 #10427 +/- ##
================================================
+ Coverage 86.13% 86.15% +0.01%
================================================
Files 139 139
Lines 22460 22438 -22
================================================
- Hits 19347 19332 -15
+ Misses 3113 3106 -7
Continue to review full report at Codecov.
|
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.
Small nits but otherwise lgtm.
} | ||
|
||
/** | ||
* Merge the partial sets produced by multiple CollectSetAggregations. Each null/nan value will be regarded as |
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.
* Merge the partial sets produced by multiple CollectSetAggregations. Each null/nan value will be regarded as | |
* Merge the partial sets produced by multiple CollectSetAggregations. Each null/NaN value will be regarded as |
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.
Fixed.
@gpucibot merge |
Exposes public APIs for collect operations as
ReductionAggregation
, which are essential to spark-rapids. In addition, this PR also extends the test framework of Reduction to discriminate output types from input types.