Use matrix to test java 8 and 17 with GH actions

prestodb · Oct 15, 2024 · 6b1917b · 6b1917b
1 parent b8617ef
commit 6b1917b
Show file tree

Hide file tree

Showing 10 changed files with 53 additions and 64 deletions.
diff --git a/.github/workflows/cleanup.yml b/.github/workflows/cleanup.yml
diff --git a/.github/workflows/hive-tests.yml b/.github/workflows/hive-tests.yml
@@ -31,12 +31,15 @@ jobs:
             - '!presto-docs/**'
 
   hive-tests:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-hive-tests-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-hive-tests-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -45,7 +48,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2
@@ -83,12 +86,15 @@ jobs:
           fi
 
   hive-dockerized-tests:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 20
     concurrency:
-      group: ${{ github.workflow }}-hive-dockerized-tests-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-hive-dockerized-tests-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -97,7 +103,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/kudu.yml b/.github/workflows/kudu.yml
@@ -29,12 +29,15 @@ jobs:
           codechange:
             - '!presto-docs/**'
   kudu:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-kudu-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-kudu-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -43,7 +46,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/maven-checks.yml b/.github/workflows/maven-checks.yml
@@ -12,10 +12,13 @@ env:
 
 jobs:
   maven-checks:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     timeout-minutes: 45
     concurrency:
-      group: ${{ github.workflow }}-maven-checks-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-maven-checks-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - name: Free Disk Space
@@ -29,7 +32,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/product-tests-basic-environment.yml b/.github/workflows/product-tests-basic-environment.yml
@@ -29,12 +29,15 @@ jobs:
           codechange:
             - '!presto-docs/**'
   product-tests-basic-environment:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-product-tests-basic-environment-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-product-tests-basic-environment-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - name: Free Disk Space
@@ -49,7 +52,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/product-tests-specific-environment.yml b/.github/workflows/product-tests-specific-environment.yml
@@ -29,12 +29,15 @@ jobs:
           codechange:
             - '!presto-docs/**'
   product-tests-specific-environment1:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-product-tests-specific-environment1-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-product-tests-specific-environment1-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - name: Free Disk Space
@@ -49,7 +52,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2
@@ -80,12 +83,15 @@ jobs:
         run: presto-product-tests/bin/run_on_docker.sh multinode-tls-kerberos -g cli,group-by,join,tls
 
   product-tests-specific-environment2:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-product-tests-specific-environment2-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-product-tests-specific-environment2-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - name: Free Disk Space
@@ -100,7 +106,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/singlestore-tests.yml b/.github/workflows/singlestore-tests.yml
@@ -30,13 +30,16 @@ jobs:
             codechange:
               - '!presto-docs/**'
   singlestore-dockerized-tests:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
 
     timeout-minutes: 30
     concurrency:
-      group: ${{ github.workflow }}-singlestore-dockerized-tests-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-singlestore-dockerized-tests-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -56,7 +59,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/spark-integration.yml b/.github/workflows/spark-integration.yml
@@ -30,12 +30,15 @@ jobs:
           codechange:
             - '!presto-docs/**'
   spark-integration:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-spark-integration-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-spark-integration-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -44,7 +47,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/test-other-modules.yml b/.github/workflows/test-other-modules.yml
@@ -30,12 +30,15 @@ jobs:
           codechange:
             - '!presto-docs/**'
   test-other-modules:
+    strategy:
+      matrix:
+        java: [ 8, 17 ]
     runs-on: ubuntu-latest
     needs: changes
     if: needs.changes.outputs.codechange == 'true'
     timeout-minutes: 60
     concurrency:
-      group: ${{ github.workflow }}-test-other-modules-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-test-other-modules-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -44,7 +47,7 @@ jobs:
       - uses: actions/setup-java@v2
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         id: cache-maven
         uses: actions/cache@v2

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -36,6 +36,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        java: [8, 17]
         modules:
           - ":presto-tests -P presto-tests-execution-memory"
           - ":presto-tests -P presto-tests-general"
@@ -68,7 +69,7 @@ jobs:
           - ":presto-iceberg"
     timeout-minutes: 80
     concurrency:
-      group: ${{ github.workflow }}-test-${{ matrix.modules }}-${{ github.event.pull_request.number }}
+      group: ${{ github.workflow }}-test-${{ matrix.modules }}-${{ github.event.pull_request.number }}-${{ matrix.java }}
       cancel-in-progress: true
     steps:
       - uses: actions/checkout@v4
@@ -79,7 +80,7 @@ jobs:
         if: needs.changes.outputs.codechange == 'true'
         with:
           distribution: 'temurin'
-          java-version: 17
+          java-version: ${{ matrix.java }}
       - name: Cache local Maven repository
         if: needs.changes.outputs.codechange == 'true'
         id: cache-maven