Merge branch-25.10 into main [skip ci] (#13589)

Change version to 25.10.0

Note: merge this PR with **Create a merge commit to merge**

Author: NvTimLiu

.github/PULL_REQUEST_TEMPLATE.md

@@ -28,15 +28,12 @@ Please provide a description of the changes proposed in this pull request. Here
 
 ### Checklists
 
-<!-- Check the items below by putting "x" in the brackets for what is done. Not all of these items may be relevant to every PR. -->
-
-This PR has:
+<!-- 
+Check the items below by putting "x" in the brackets for what is done.
+Not all of these items may be relevant to every PR, so please check only those that apply.
+-->
 
-- [ ] added documentation for new or modified features or behaviors.
-- [ ] updated the license in the source code files when it is required.
-- [ ] added new tests or modified existing tests to cover new code paths.
+- [ ] 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.)
-
-Please select one of the following options:
-- [ ] Performance testing has been performed and its results are added in the PR description.
-- [ ] An issue is filed for performance testing and its link is added in the PR description. (Select this if performance testing will not be completed before the PR is submitted.)
+- [ ] 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.

.github/workflows/shell-check.yml

@@ -0,0 +1,31 @@
+# Copyright (c) 2025, NVIDIA CORPORATION.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+# A workflow to check shell script syntax
+name: shell check
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened]
+
+jobs:
+  shell-check:
+    runs-on: ubuntu-latest
+    if: "!contains(github.event.pull_request.title, '[bot]')"
+    steps:
+      - name: Checkout code
+        uses: NVIDIA/spark-rapids-common/checkout@main
+
+      - name: Run ShellCheck
+        uses: NVIDIA/spark-rapids-common/shell-check@main

CHANGELOG.md

@@ -127,15 +127,15 @@ mvn -pl dist -PnoSnapshots package -DskipTests
 Verify that shim-specific classes are hidden from a conventional classloader.
 
 ```bash
-$ javap -cp dist/target/rapids-4-spark_2.12-25.08.0-cuda12.jar com.nvidia.spark.rapids.shims.SparkShimImpl
+$ javap -cp dist/target/rapids-4-spark_2.12-25.10.0-cuda12.jar com.nvidia.spark.rapids.shims.SparkShimImpl
 Error: class not found: com.nvidia.spark.rapids.shims.SparkShimImpl
 ```
 
 However, its bytecode can be loaded if prefixed with `spark3XY` not contained in the package name
 
 ```bash
-$ javap -cp dist/target/rapids-4-spark_2.12-25.08.0-cuda12.jar spark320.com.nvidia.spark.rapids.shims.SparkShimImpl | head -2
-Warning: File dist/target/rapids-4-spark_2.12-25.08.0-cuda12.jar(/spark320/com/nvidia/spark/rapids/shims/SparkShimImpl.class) does not contain class spark320.com.nvidia.spark.rapids.shims.SparkShimImpl
+$ javap -cp dist/target/rapids-4-spark_2.12-25.10.0-cuda12.jar spark320.com.nvidia.spark.rapids.shims.SparkShimImpl | head -2
+Warning: File dist/target/rapids-4-spark_2.12-25.10.0-cuda12.jar(/spark320/com/nvidia/spark/rapids/shims/SparkShimImpl.class) does not contain class spark320.com.nvidia.spark.rapids.shims.SparkShimImpl
 Compiled from "SparkShims.scala"
 public final class com.nvidia.spark.rapids.shims.SparkShimImpl {
 ```
@@ -177,7 +177,7 @@ mvn package -pl dist -am -Dbuildver=340 -DallowConventionalDistJar=true
 Verify `com.nvidia.spark.rapids.shims.SparkShimImpl` is conventionally loadable:
 
 ```bash
-$ javap -cp dist/target/rapids-4-spark_2.12-25.08.0-cuda12.jar com.nvidia.spark.rapids.shims.SparkShimImpl | head -2
+$ javap -cp dist/target/rapids-4-spark_2.12-25.10.0-cuda12.jar com.nvidia.spark.rapids.shims.SparkShimImpl | head -2
 Compiled from "SparkShims.scala"
 public final class com.nvidia.spark.rapids.shims.SparkShimImpl {
 ```

CONTRIBUTING.md

@@ -75,7 +75,7 @@ as a `provided` dependency.
 <dependency>
     <groupId>com.nvidia</groupId>
     <artifactId>rapids-4-spark_2.12</artifactId>
-    <version>25.08.0</version>
+    <version>25.10.0</version>
     <scope>provided</scope>
 </dependency>
 ```

README.md

@@ -22,13 +22,13 @@
     <parent>
         <groupId>com.nvidia</groupId>
         <artifactId>rapids-4-spark-jdk-profiles_2.12</artifactId>
-        <version>25.08.0</version>
+        <version>25.10.0</version>
         <relativePath>../jdk-profiles/pom.xml</relativePath>
     </parent>
     <artifactId>rapids-4-spark-aggregator_2.12</artifactId>
     <name>RAPIDS Accelerator for Apache Spark Aggregator</name>
     <description>Creates an aggregated shaded package of the RAPIDS plugin for Apache Spark</description>
-    <version>25.08.0</version>
+    <version>25.10.0</version>
 
     <properties>
         <rapids.module>aggregator</rapids.module>

aggregator/pom.xml

@@ -22,11 +22,11 @@
     <parent>
         <groupId>com.nvidia</groupId>
         <artifactId>rapids-4-spark-shim-deps-parent_2.12</artifactId>
-        <version>25.08.0</version>
+        <version>25.10.0</version>
         <relativePath>../shim-deps/pom.xml</relativePath>
     </parent>
     <artifactId>rapids-4-spark-api-validation_2.12</artifactId>
-    <version>25.08.0</version>
+    <version>25.10.0</version>
 
     <properties>
         <rapids.module>api_validation</rapids.module>

api_validation/pom.xml

@@ -75,7 +75,7 @@ function bloopInstall() {
       rm -rf $bloopDir
       mv $bloopTmpConfigDir $bloopDir
       echo "generated bloop files under $bloopDir"
-      bloopDirsGenerated+=($bloopDir)
+      bloopDirsGenerated+=("${bloopDir}")
     done
     echo "#### Created bloop projects ${bloopDirsGenerated[@]}"
     echo "Execute"

build/buildall

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 #
-# Copyright (c) 2020-2024, NVIDIA CORPORATION. All rights reserved.
+# Copyright (c) 2020-2025, NVIDIA CORPORATION. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -46,13 +46,13 @@ if [ ! -f "$JDEST" ]; then
 fi
 
 if [ "$#" != "0" ]; then
-  ARGS=$@
+  ARGS=("$@")
 else
-  ARGS=$(find . -name "jacoco.exec" -printf "%p ")
+  ARGS=($(find . -name "jacoco.exec" -printf "%p "))
 fi
 
 mkdir -p $XML_LOC
 # run the report
-java -jar "$JDEST" report --classfiles "$TMP_CLASS" --html "$HTML_LOC" --xml "$XML_LOC/jacoco.xml" $SOURCE_WITH_ARGS $ARGS
+java -jar "$JDEST" report --classfiles "$TMP_CLASS" --html "$HTML_LOC" --xml "$XML_LOC/jacoco.xml" $SOURCE_WITH_ARGS "${ARGS[@]}"
 
 echo "The report is at file://"$(readlink -f "$HTML_LOC"/index.html)

build/coverage-report

@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 #
-# Copyright (c) 2023-2024, NVIDIA CORPORATION. All rights reserved.
+# Copyright (c) 2023-2025, NVIDIA CORPORATION. All rights reserved.
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -53,7 +53,7 @@ fi
 TO_VERSION=$1
 
 check_scala_version() {
-  for i in ${VALID_VERSIONS[*]}; do [ $i = "$1" ] && return 0; done
+  for i in "${VALID_VERSIONS[@]}"; do [ $i = "$1" ] && return 0; done
   echo "Invalid Scala version: $1. Valid versions: ${VALID_VERSIONS[*]}" 1>&2
   exit 1
 }