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 a6088e5  [SPARK-35924][BUILD][TESTS] Add Java 17 ea build test to 
GitHub action
a6088e5 is described below

commit a6088e50363a2a39c2f14287b82d5e84a0efb53b
Author: William Hyun <will...@apache.org>
AuthorDate: Tue Jun 29 11:19:38 2021 -0700

    [SPARK-35924][BUILD][TESTS] Add Java 17 ea build test to GitHub action
    
    ### What changes were proposed in this pull request?
    This PR aims to add Java 17-ea build test to GitHub action.
    
    ### Why are the changes needed?
    To improve test coverage.
    
    ### Does this PR introduce _any_ user-facing change?
    No.
    
    ### How was this patch tested?
    Pass newly added Java 17-ea GitHub action job.
    
    Closes #33126 from williamhyun/SPARK-35924.
    
    Authored-by: William Hyun <will...@apache.org>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .github/workflows/build_and_test.yml | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/build_and_test.yml 
b/.github/workflows/build_and_test.yml
index d244135..570983d 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -414,8 +414,14 @@ jobs:
         cd docs
         bundle exec jekyll build
 
-  java-11:
-    name: Java 11 build with Maven
+  java-11-17:
+    name: Java ${{ matrix.java }} build with Maven
+    strategy:
+      fail-fast: false
+      matrix:
+        java:
+          - 11
+          - 17-ea
     runs-on: ubuntu-20.04
     steps:
     - name: Checkout Spark repository
@@ -446,19 +452,20 @@ jobs:
       uses: actions/cache@v2
       with:
         path: ~/.m2/repository
-        key: java11-maven-${{ hashFiles('**/pom.xml') }}
+        key: java${{ matrix.java }}-maven-${{ hashFiles('**/pom.xml') }}
         restore-keys: |
-          java11-maven-
-    - name: Install Java 11
+          java${{ matrix.java }}-maven-
+    - name: Install Java ${{ matrix.java }}
       uses: actions/setup-java@v1
       with:
-        java-version: 11
+        java-version: ${{ matrix.java }}
     - name: Build with Maven
       run: |
         export MAVEN_OPTS="-Xss64m -Xmx2g -XX:ReservedCodeCacheSize=1g 
-Dorg.slf4j.simpleLogger.defaultLogLevel=WARN"
         export MAVEN_CLI_OPTS="--no-transfer-progress"
+        export JAVA_VERSION=${{ matrix.java }}
         # It uses Maven's 'install' intentionally, see 
https://github.com/apache/spark/pull/26414.
-        ./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pmesos -Pkubernetes 
-Phive -Phive-thriftserver -Phadoop-cloud -Djava.version=11 install
+        ./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pmesos -Pkubernetes 
-Phive -Phive-thriftserver -Phadoop-cloud -Djava.version=${JAVA_VERSION/-ea} 
install
         rm -rf ~/.m2/repository/org/apache/spark
 
   scala-213:

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

Reply via email to