Skip to content

[auto-merge] branch-25.10 to branch-25.12 [skip ci] [bot]#13529

Merged
nvauto merged 1 commit into
branch-25.12from
branch-25.10
Sep 29, 2025
Merged

[auto-merge] branch-25.10 to branch-25.12 [skip ci] [bot]#13529
nvauto merged 1 commit into
branch-25.12from
branch-25.10

Conversation

@nvauto

@nvauto nvauto commented Sep 29, 2025

Copy link
Copy Markdown
Collaborator

auto-merge triggered by github actions on branch-25.10 to create a PR keeping branch-25.12 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

This changes the plugin to call the new spark-rapids-jni API for
rounding rather than the old cudf API. The rounding of floating point
functionality will be removed from cudf so we've moved the float
rounding (and the API) into spark-rapids-jni.

This is part of resolving [sr-jni issue
3585](NVIDIA/cudf-spark-jni#3585)

[This PR](rapidsai/cudf#20110) removes the old
java code from cudf, and will be merged in cudf 25.12.
[This PR](NVIDIA/cudf-spark-jni#3770) moves
the floating rounding into spark-rapids-jni and changes the API. This
will be merged in 25.10 and is a dependency of this PR.

Note that I have tested this code applying both of the above PRs and all
of the spark-rapids integration tests pass.

### Checklists

- [ ] This PR has added documentation for new or modified features or
behaviors.
- [ ] This PR has added new tests or modified existing tests to cover
new code paths.
(Please explain in the PR description how the new code paths are tested,
such as names of the new/existing tests that cover them.)
- [ ] Performance testing has been performed and its results are added
in the PR description. Or, an issue has been filed with a link in the PR
description.

---------

Signed-off-by: Paul Mattione <pmattione@nvidia.com>
@nvauto nvauto merged commit d3271fc into branch-25.12 Sep 29, 2025
@nvauto

nvauto commented Sep 29, 2025

Copy link
Copy Markdown
Collaborator Author

SUCCESS - auto-merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants