Skip to content

Rename RAPIDS Accelerator for Apache Spark to NVIDIA cuDF plugin for Apache Spark - #5024

Open
sameerz wants to merge 15 commits into
NVIDIA:mainfrom
sameerz:cudf-spark-rename-1
Open

Rename RAPIDS Accelerator for Apache Spark to NVIDIA cuDF plugin for Apache Spark#5024
sameerz wants to merge 15 commits into
NVIDIA:mainfrom
sameerz:cudf-spark-rename-1

Conversation

@sameerz

@sameerz sameerz commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Rename RAPIDS Accelerator for Apache Spark to NVIDIA cuDF plugin for Apache Spark.

Update first reference to the product to NVIDIA cuDF plugin for Apache Spark, and subsequent references to the cuDF plugin. It is ok to include JNI in the name for this repository, so NVIDIA cuDF plugin JNI for Apache Spark.

Update github references for cudf, cudf-spark, cudf-spark-jni

@sameerz
Rename RAPIDS Accelerator for Apache Spark to NVIDIA cuDF plugin for …
94aa3e8 
…Apache Spark

Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz sameerz added the documentation Improvements or additions to documentation label Aug 19, 2026
@sameerz
sameerz requested a review from a team as a code owner August 19, 2026 23:35
@greptile-apps

greptile-apps Bot commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR renames product and repository references from RAPIDS Accelerator for Apache Spark to NVIDIA cuDF plugin for Apache Spark while retaining existing artifact and binary-contract names.

  • Updates documentation, templates, links, Maven metadata, diagnostics, and automation identities.
  • Advances the cuDF submodule revision.
  • Leaves Maven coordinates and the profiler stream magic unchanged for compatibility.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
README.md Introduces the new product name and documents compatibility redirects and retained artifact names.
CONTRIBUTING.md Updates contributor-facing product names, repository paths, and cross-repository workflow links.
pom.xml Renames project display metadata and SCM URLs without changing Maven artifact coordinates or dependencies.
.github/workflows/sync-clang-format-version.yml Renames comments and automated commit attribution without altering workflow control flow.
src/main/cpp/profiler/spark_rapids_profile_converter.cpp Updates user-facing product terminology while preserving the existing profile-format magic contract.
thirdparty/cudf Advances the pinned cuDF revision; no concrete compatibility failure was established from the available evidence.

Reviews (7): Last reviewed commit: "Merge remote-tracking branch 'origin/cud..." | Re-trigger Greptile

nvliyuan
nvliyuan reviewed Aug 20, 2026
View reviewed changes
Comment thread CODE_OF_CONDUCT.md Outdated
@nvliyuan

nvliyuan commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Some user-visible references to the old product/repository names.

  • src/main/cpp/faultinj/README.md:25
  • src/main/cpp/profiler/spark_rapids_profile_converter.cpp:117,369
  • src/main/cpp/src/NVMLJni.cpp:24
  • src/main/cpp/tests/hash.cpp:205,620

Old issue URL:

  • bloom_filter.cu:201
  • cast_string_to_datetime.cu:558
  • from_json_to_structs.cu:546
  • ParseURITest.java:302,362
  • IcebergTruncateTest.java:52

@nvliyuan

nvliyuan commented Aug 20, 2026

Copy link
Copy Markdown
Contributor
  • ci/submodule-sync.sh

still creates commits as “spark-rapids automation”, while sync-clang-format-version.yml now uses “cudf-spark automation”. These should be aligned.

nvliyuan
nvliyuan reviewed Aug 20, 2026
View reviewed changes
Comment thread README.md Outdated
nvliyuan
nvliyuan suggested changes Aug 20, 2026
View reviewed changes
Comment thread README.md Outdated
@sameerz
updated additional spark-rapids references
67a0890 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
sameerz requested a review from nvliyuan August 20, 2026 05:20
@nvliyuan

nvliyuan commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

LTGM

sameerz and others added 2 commits August 20, 2026 08:08
@sameerz
Update license
4be8bc9 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
Merge branch 'main' into cudf-spark-rename-1
9f4d1c7
gerashegalov
gerashegalov reviewed Aug 22, 2026
View reviewed changes

@gerashegalov gerashegalov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

New trailing whitespace fails pre-commit CI.
CONTRIBUTING.md:67 and profiler/README.md:4
fails precommit

The repository-link rename remains incomplete.
from_json_to_raw_map.cu:188 still links to github.com/rapidsai/cudf.
Additional old shorthand references remain in CMakeLists.txt:53, cast_string_to_float.cu:195,396, and timezones.cu:537.

Update these to NVIDIA/cudf or NVIDIA/cudf-spark as applicable.

sameerz added 6 commits August 27, 2026 21:34
@sameerz
resolve merge conflicts
96b212a 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
updated additional spark-rapids references
9e5de25 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
Update license
cfec65f 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
fix trailing spaces; fix some references to rapidsai
db14189 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
resolve merge conflicts
488ec1d 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
Fix clang-format violations
a8bd7fd 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz

sameerz commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator Author

build

@sameerz
sameerz requested a review from gerashegalov August 28, 2026 04:57
@sameerz

sameerz commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator Author

Build blocked on #5014

gerashegalov
gerashegalov reviewed Aug 28, 2026
View reviewed changes
Comment thread pom.xml Outdated
gerashegalov
gerashegalov reviewed Aug 28, 2026
View reviewed changes
Comment thread src/main/cpp/CMakeLists.txt
gerashegalov
gerashegalov reviewed Aug 28, 2026
View reviewed changes
Comment thread CONTRIBUTING.md
@sameerz
address comments
0025c09 
Signed-off-by: Sameer Raheja <sraheja@nvidia.com>
@sameerz
sameerz requested a review from gerashegalov August 29, 2026 06:27
gerashegalov
gerashegalov reviewed Aug 29, 2026
View reviewed changes
Comment thread .pre-commit-config.yaml
if [ -z "$our_ver" ] || [ -z "$cudf_ver" ] || [ "$our_ver" != "$cudf_ver" ]; then
echo "ERROR: clang-format version mismatch or failed to extract version!";
echo " spark-rapids-jni: ${our_ver:-(empty)}";
echo " cudf-spark-jni: ${our_ver:-(empty)}";

@gerashegalov gerashegalov Aug 29, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Revert the change to the submodule. This will be a cudf PR.

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

Reviewers

@gerashegalov gerashegalov gerashegalov left review comments
+1 more reviewer
@nvliyuan nvliyuan nvliyuan approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sameerz @nvliyuan @gerashegalov @nvauto