This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 4faae1d88e8d [SPARK-52728][BUILD] Remove the usage of non-existent 
profiles
4faae1d88e8d is described below

commit 4faae1d88e8db5da881c3aa9bba29724d998f349
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Wed Jul 9 11:13:10 2025 -0700

    [SPARK-52728][BUILD] Remove the usage of non-existent profiles
    
    ### What changes were proposed in this pull request?
    This PR removes the usage of non-existent profiles `noshade-connect` and 
`noshade-protobuf` in `SparkBuild.scala`.
    
    ### Why are the changes needed?
    Profiles that have never existed in the Maven `pom.xml` should not be used 
in `SparkBuild.scala`.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    - Pass Github Actions
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #51418 from LuciferYang/SPARK-52728.
    
    Authored-by: yangjie01 <yangji...@baidu.com>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 project/SparkBuild.scala | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/project/SparkBuild.scala b/project/SparkBuild.scala
index ff75142457e9..8a902b9b0cf3 100644
--- a/project/SparkBuild.scala
+++ b/project/SparkBuild.scala
@@ -1530,18 +1530,15 @@ object CopyDependencies {
             destJar.delete()
           }
 
-          if (jar.getName.contains("spark-connect-common") &&
-            !SbtPomKeys.profiles.value.contains("noshade-connect")) {
+          if (jar.getName.contains("spark-connect-common")) {
             // Don't copy the spark connect common JAR as it is shaded in the 
spark connect.
           } else if (jar.getName.contains("connect-client-jvm")) {
             // Do not place Spark Connect client jars as it is not built-in.
           } else if (noProvidedSparkJars && 
jar.getName.contains("spark-avro")) {
             // Do not place Spark Avro jars as it is not built-in.
-          } else if (jar.getName.contains("spark-connect") &&
-            !SbtPomKeys.profiles.value.contains("noshade-connect")) {
+          } else if (jar.getName.contains("spark-connect")) {
             Files.copy(fid.toPath, destJar.toPath)
-          } else if (jar.getName.contains("spark-protobuf") &&
-            !SbtPomKeys.profiles.value.contains("noshade-protobuf")) {
+          } else if (jar.getName.contains("spark-protobuf")) {
             if (!noProvidedSparkJars) {
               Files.copy(fidProtobuf.toPath, destJar.toPath)
             }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to