Skip to content

Update dependency version JNI, private, hybrid to 25.12.0 - #13988

Merged
pxLi merged 1 commit into
release/25.12from
update-dependency
Dec 15, 2025
Merged

Update dependency version JNI, private, hybrid to 25.12.0#13988
pxLi merged 1 commit into
release/25.12from
update-dependency

Conversation

@nvauto

@nvauto nvauto commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator

Wait for the pre-merge CI job to SUCCEED

Wait for the pre-merge CI job to SUCCEED

Signed-off-by: nvauto <70000568+nvauto@users.noreply.github.com>
@nvauto
nvauto requested a review from a team as a code owner December 10, 2025 03:42
@nvauto

nvauto commented Dec 10, 2025

Copy link
Copy Markdown
Collaborator Author

build

@greptile-apps

greptile-apps Bot commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

This PR updates the dependency versions for three NVIDIA RAPIDS libraries (spark-rapids-jni, spark-rapids-private, and spark-rapids-hybrid) from 25.12.0-SNAPSHOT to the stable 25.12.0 release. The change is applied consistently across both pom.xml (Scala 2.12) and scala2.13/pom.xml (Scala 2.13) build configurations.

  • Updated spark-rapids-jni.version from 25.12.0-SNAPSHOT to 25.12.0
  • Updated spark-rapids-private.version from 25.12.0-SNAPSHOT to 25.12.0
  • Updated spark-rapids-hybrid.version from 25.12.0-SNAPSHOT to 25.12.0

This is a standard dependency version bump from snapshot to release, following the same pattern as previous releases (e.g., the 25.10.0 update in PR #13587).

Confidence Score: 5/5

  • This PR is safe to merge - it's a standard dependency version bump from SNAPSHOT to release with no code changes.
  • Score of 5 reflects: (1) minimal scope - only Maven property version strings updated, (2) follows established pattern from previous releases, (3) both Scala 2.12 and 2.13 pom files are updated consistently, (4) no logic changes or new code introduced.
  • No files require special attention.

Important Files Changed

File Analysis

Filename Score Overview
pom.xml 5/5 Updated dependency versions for spark-rapids-jni, spark-rapids-private, and spark-rapids-hybrid from 25.12.0-SNAPSHOT to 25.12.0 release version.
scala2.13/pom.xml 5/5 Updated dependency versions for spark-rapids-jni, spark-rapids-private, and spark-rapids-hybrid from 25.12.0-SNAPSHOT to 25.12.0 release version (Scala 2.13 variant).

Sequence Diagram

sequenceDiagram
    participant Build as Maven Build
    participant Parent as pom.xml
    participant Scala213 as scala2.13/pom.xml
    participant JNI as spark-rapids-jni
    participant Private as spark-rapids-private
    participant Hybrid as spark-rapids-hybrid

    Note over Parent,Scala213: Version property update
    Parent->>Parent: spark-rapids-jni.version = 25.12.0
    Parent->>Parent: spark-rapids-private.version = 25.12.0
    Parent->>Parent: spark-rapids-hybrid.version = 25.12.0
    Scala213->>Scala213: spark-rapids-jni.version = 25.12.0
    Scala213->>Scala213: spark-rapids-private.version = 25.12.0
    Scala213->>Scala213: spark-rapids-hybrid.version = 25.12.0

    Note over Build,Hybrid: Dependency resolution
    Build->>JNI: Fetch 25.12.0 (release)
    Build->>Private: Fetch 25.12.0 (release)
    Build->>Hybrid: Fetch 25.12.0 (release)
Loading

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@NvTimLiu
NvTimLiu marked this pull request as draft December 10, 2025 06:12
@NvTimLiu NvTimLiu self-assigned this Dec 10, 2025
@NvTimLiu NvTimLiu added cudf_dependency An issue or PR with this label depends on a new feature in cudf build Related to CI / CD or cleanly building labels Dec 10, 2025
@NvTimLiu

Copy link
Copy Markdown
Collaborator

Draft this change until the dependency jars are released to maven central.

@pxLi

pxLi commented Dec 12, 2025

Copy link
Copy Markdown
Member

build

@pxLi
pxLi marked this pull request as ready for review December 15, 2025 00:55
@pxLi
pxLi merged commit 9f893d9 into release/25.12 Dec 15, 2025
67 of 122 checks passed

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@nvauto
nvauto deleted the update-dependency branch January 26, 2026 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Related to CI / CD or cleanly building cudf_dependency An issue or PR with this label depends on a new feature in cudf

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants