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

jackylee pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 8683d66f6f [VL] Remove variable EXTRA_FLAGS in velox_backend_x86.yml 
(#10274)
8683d66f6f is described below

commit 8683d66f6fb95ac86e15cd69b95d8ac611057fd9
Author: Hongze Zhang <[email protected]>
AuthorDate: Wed Jul 30 05:53:17 2025 +0100

    [VL] Remove variable EXTRA_FLAGS in velox_backend_x86.yml (#10274)
---
 .github/workflows/velox_backend_arm.yml            | 18 ----------------
 .../workflows/velox_backend_enhanced_features.yml  | 20 +-----------------
 .github/workflows/velox_backend_x86.yml            | 24 +++-------------------
 3 files changed, 4 insertions(+), 58 deletions(-)

diff --git a/.github/workflows/velox_backend_arm.yml 
b/.github/workflows/velox_backend_arm.yml
index 444796ae6f..b9e4cae510 100644
--- a/.github/workflows/velox_backend_arm.yml
+++ b/.github/workflows/velox_backend_arm.yml
@@ -43,24 +43,6 @@ env:
   WGET_CMD: 'wget -nv'
   SETUP: 'bash .github/workflows/util/setup_helper.sh'
   CCACHE_DIR: "${{ github.workspace }}/.ccache"
-  # for JDK17 unit tests
-  EXTRA_FLAGS:  "-XX:+IgnoreUnrecognizedVMOptions 
-                --add-opens=java.base/java.lang=ALL-UNNAMED
-                --add-opens=java.base/java.lang.invoke=ALL-UNNAMED
-                --add-opens=java.base/java.lang.reflect=ALL-UNNAMED
-                --add-opens=java.base/java.io=ALL-UNNAMED
-                --add-opens=java.base/java.net=ALL-UNNAMED
-                --add-opens=java.base/java.nio=ALL-UNNAMED
-                --add-opens=java.base/java.util=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED
-                --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.ch=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.cs=ALL-UNNAMED
-                --add-opens=java.base/sun.security.action=ALL-UNNAMED
-                --add-opens=java.base/sun.util.calendar=ALL-UNNAMED
-                -Djdk.reflect.useDirectMethodHandle=false
-                -Dio.netty.tryReflectionSetAccessible=true"
 
 concurrency:
   group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ 
github.workflow }}
diff --git a/.github/workflows/velox_backend_enhanced_features.yml 
b/.github/workflows/velox_backend_enhanced_features.yml
index 228f5abd22..5ba5c785ec 100644
--- a/.github/workflows/velox_backend_enhanced_features.yml
+++ b/.github/workflows/velox_backend_enhanced_features.yml
@@ -43,24 +43,6 @@ env:
   WGET_CMD: 'wget -nv'
   SETUP: 'bash .github/workflows/util/setup_helper.sh'
   CCACHE_DIR: "${{ github.workspace }}/.ccache"
-  # for JDK17 unit tests
-  EXTRA_FLAGS:  "-XX:+IgnoreUnrecognizedVMOptions 
-                --add-opens=java.base/java.lang=ALL-UNNAMED
-                --add-opens=java.base/java.lang.invoke=ALL-UNNAMED
-                --add-opens=java.base/java.lang.reflect=ALL-UNNAMED
-                --add-opens=java.base/java.io=ALL-UNNAMED
-                --add-opens=java.base/java.net=ALL-UNNAMED
-                --add-opens=java.base/java.nio=ALL-UNNAMED
-                --add-opens=java.base/java.util=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED
-                --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.ch=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.cs=ALL-UNNAMED
-                --add-opens=java.base/sun.security.action=ALL-UNNAMED
-                --add-opens=java.base/sun.util.calendar=ALL-UNNAMED
-                -Djdk.reflect.useDirectMethodHandle=false
-                -Dio.netty.tryReflectionSetAccessible=true"
 
 concurrency:
   group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ 
github.workflow }}
@@ -146,7 +128,7 @@ jobs:
           ls -l $SPARK_HOME
           $MVN_CMD clean test -Pspark-3.4 -Pjava-17 -Pbackends-velox -Piceberg 
-Pdelta -Phudi \
           
-DtagsToExclude=org.apache.spark.tags.ExtendedSQLTest,org.apache.gluten.tags.UDFTest,org.apache.gluten.tags.SkipTest
 \
-          -DargLine="-Dspark.test.home=$SPARK_HOME ${EXTRA_FLAGS}"
+          -DargLine="-Dspark.test.home=$SPARK_HOME"
       - name: Upload test report
         if: always()
         uses: actions/upload-artifact@v4
diff --git a/.github/workflows/velox_backend_x86.yml 
b/.github/workflows/velox_backend_x86.yml
index 9f695a634b..901395c816 100644
--- a/.github/workflows/velox_backend_x86.yml
+++ b/.github/workflows/velox_backend_x86.yml
@@ -43,24 +43,6 @@ env:
   WGET_CMD: 'wget -nv'
   SETUP: 'bash .github/workflows/util/setup_helper.sh'
   CCACHE_DIR: "${{ github.workspace }}/.ccache"
-  # for JDK17 unit tests
-  EXTRA_FLAGS:  "-XX:+IgnoreUnrecognizedVMOptions 
-                --add-opens=java.base/java.lang=ALL-UNNAMED
-                --add-opens=java.base/java.lang.invoke=ALL-UNNAMED
-                --add-opens=java.base/java.lang.reflect=ALL-UNNAMED
-                --add-opens=java.base/java.io=ALL-UNNAMED
-                --add-opens=java.base/java.net=ALL-UNNAMED
-                --add-opens=java.base/java.nio=ALL-UNNAMED
-                --add-opens=java.base/java.util=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent=ALL-UNNAMED
-                --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED
-                --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.ch=ALL-UNNAMED
-                --add-opens=java.base/sun.nio.cs=ALL-UNNAMED
-                --add-opens=java.base/sun.security.action=ALL-UNNAMED
-                --add-opens=java.base/sun.util.calendar=ALL-UNNAMED
-                -Djdk.reflect.useDirectMethodHandle=false
-                -Dio.netty.tryReflectionSetAccessible=true"
 
 concurrency:
   group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ 
github.workflow }}
@@ -840,7 +822,7 @@ jobs:
           ls -l $SPARK_HOME
           $MVN_CMD clean test -Pspark-3.4 -Pjava-17 -Pbackends-velox -Piceberg 
-Pdelta -Phudi -Pspark-ut \
           
-DtagsToExclude=org.apache.spark.tags.ExtendedSQLTest,org.apache.gluten.tags.UDFTest,org.apache.gluten.tags.EnhancedFeaturesTest,org.apache.gluten.tags.SkipTest
 \
-          -DargLine="-Dspark.test.home=$SPARK_HOME ${EXTRA_FLAGS}"
+          -DargLine="-Dspark.test.home=$SPARK_HOME"
       - name: Upload test report
         if: always()
         uses: actions/upload-artifact@v4
@@ -890,7 +872,7 @@ jobs:
           ls -l $SPARK_HOME
           $MVN_CMD clean test -Pspark-3.4 -Pjava-17 -Pbackends-velox -Piceberg 
-Pdelta -Pspark-ut -Phudi \
           -DtagsToInclude=org.apache.spark.tags.ExtendedSQLTest \
-          -DargLine="-Dspark.test.home=$SPARK_HOME ${EXTRA_FLAGS}"
+          -DargLine="-Dspark.test.home=$SPARK_HOME"
       - name: Upload test report
         if: always()
         uses: actions/upload-artifact@v4
@@ -939,7 +921,7 @@ jobs:
           export PATH=$JAVA_HOME/bin:$PATH
           java -version
           $MVN_CMD clean test -Pspark-3.5 -Pjava-17 -Pbackends-velox -Piceberg 
-Pdelta -Phudi -Pspark-ut \
-          -DargLine="-Dspark.test.home=/opt/shims/spark35/spark_home/  
${EXTRA_FLAGS}" \
+          -DargLine="-Dspark.test.home=/opt/shims/spark35/spark_home/" \
           
-DtagsToExclude=org.apache.spark.tags.ExtendedSQLTest,org.apache.gluten.tags.UDFTest,org.apache.gluten.tags.EnhancedFeaturesTest,org.apache.gluten.tags.SkipTest
       - name: Upload test report
         if: always()


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to