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

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


The following commit(s) were added to refs/heads/main by this push:
     new fd0cf11ec [CELEBORN-738] Enable CI for Java 17
fd0cf11ec is described below

commit fd0cf11eca2872c66bfb76cd6d873f793bf102b6
Author: Cheng Pan <[email protected]>
AuthorDate: Thu Jun 29 13:47:55 2023 +0800

    [CELEBORN-738] Enable CI for Java 17
    
    ### What changes were proposed in this pull request?
    
    Enable CI for Celeborn Master/Worker and Client with Spark 3.3/3.4
    
    ### Why are the changes needed?
    
    Ensure Celeborn works on Java 17.
    
    Note: there may be some code paths that are not covered by tests, we should 
fix them in the future.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Pass GA
    
    Closes #1649 from pan3793/CELEBORN-738.
    
    Authored-by: Cheng Pan <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 .github/workflows/maven.yml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
index c55248f25..e443ae005 100644
--- a/.github/workflows/maven.yml
+++ b/.github/workflows/maven.yml
@@ -22,6 +22,7 @@ jobs:
         java:
           - 8
           - 11
+          - 17
     steps:
     - uses: actions/checkout@v2
     - name: Setup JDK ${{ matrix.java }}
@@ -86,12 +87,21 @@ jobs:
         java:
           - 8
           - 11
+          - 17
         spark:
           - '3.0'
           - '3.1'
           - '3.2'
           - '3.3'
           - '3.4'
+        exclude:
+          # SPARK-33772: Spark supports JDK 17 since 3.3.0
+          - java: 17
+            spark: '3.0'
+          - java: 17
+            spark: '3.1'
+          - java: 17
+            spark: '3.2'
     steps:
     - uses: actions/checkout@v2
     - name: Setup JDK ${{ matrix.java }}

Reply via email to