-
Notifications
You must be signed in to change notification settings - Fork 232
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
[REVIEW] Support GpuFilter and GpuCoalesceBatches for decimal data #1192
Conversation
Signed-off-by: sperlingxx <lovedreamf@gmail.com>
705bc05
to
b04e5b1
Compare
build |
The integration tests started to crash after this change. I am not 100% sure what happened but a lot of the tests started to get errors not being able to connect to the JVM process which makes me think that it crashed. (you could try running with TEST_PARALLEL=0 in your environment to get the tests to spit out more info about the run, but it will take longer) |
You can probably also look at |
OK I am investigating it |
build |
1 similar comment
build |
build |
tests/src/test/scala/com/nvidia/spark/rapids/FilterExprSuite.scala
Outdated
Show resolved
Hide resolved
…cala Co-authored-by: Jason Lowe <jlowe@nvidia.com>
build |
build |
build |
1 similar comment
build |
rebuild |
build |
2 similar comments
build |
build |
Signed-off-by: sperlingxx <lovedreamf@gmail.com> Co-authored-by: Jason Lowe <jlowe@nvidia.com>
Signed-off-by: sperlingxx <lovedreamf@gmail.com> Co-authored-by: Jason Lowe <jlowe@nvidia.com>
…IDIA#1192) Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Since both PR to enable
cudf::apply_boolean_mask
for decimal and PR to enablecudf::contiguous_split
for decimal are ready, we can supportGpuCoalesceBatches
andGpuFilter
(which relied onGpuCoalesceBatches
).This PR is to enable
GpuFilter
for decimal type. And related tests forGpuFilter
andGpuCoalesceBatches
are also provided.In this PR,
isNotNull
is used as predicate of decimal filter because relational operators for decimal type are still unsupported.