Upgrade compiler to JDK17, fixes deephaven#1367.

Notes: * Gradle was upgraded to 6.9.1 to fix issue with needing to manually open jdk.compiler/com.sun.tools.javac.code. See gradle/gradle#15538, https://docs.gradle.org/6.9/release-notes.html, https://docs.gradle.org/6.9.1/release-notes.html * Gradle 6.9.1 can't be run from JDK17. As part of the CI setup, we need to make sure a non-JDK17 is set as the default. See https://docs.gradle.org/6.9.1/userguide/compatibility.html * zulu seems to provide more consistent release versions. It may be that we "adopt" is no longer terminilogy that should be used. See https://blog.adoptopenjdk.net/2021/03/transition-to-eclipse-an-update/ * Groovy does not like to compile the test code for open-api-lang-tools with JDK17. Various things were tried. Ended up setting Groovy compiler to 8 for now. May need to bump groovy version? * Some additional hinting was needing in the lambdas in TableHandleManagerSerial. I didn't dig in further why. * The gradle property org.gradle.java.installations.auto-download=false was removed from gradle.properties. Users wishing to guard against auto-download should set it as appropriate in ~/.gradle/gradle.properties. Note: auto-download is still disabled for CI.
devinrsmith · Oct 7, 2021 · 0ebb747 · 0ebb747
1 parent 709bf62
commit 0ebb747
Show file tree

Hide file tree

Showing 21 changed files with 255 additions and 92 deletions.
diff --git a/.github/workflows/build-ci.yml b/.github/workflows/build-ci.yml
@@ -11,14 +11,26 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      # note: this uses Zulu and not AdoptOpenJDK or other. should make sure we build and test on the same one...
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
           java-version: '8.0.302'
 
+      - name: Setup gradle properties
+        run: |
+          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
+
       - name: Get Semver
         id: semver
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}
@@ -58,11 +70,6 @@ jobs:
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Setup gradle properties
-        run: |
-          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
-
       - name: Create Dockerfile and context
         uses: burrunan/gradle-cache-action@v1
         with:
@@ -88,13 +95,26 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
           java-version: '8.0.302'
 
+      - name: Setup gradle properties
+        run: |
+          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
+
       - name: Get Semver
         id: semver
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}
@@ -133,11 +153,6 @@ jobs:
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Setup gradle properties
-        run: |
-          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
-
       - name: Create Dockerfile and context
         uses: burrunan/gradle-cache-action@v1
         with:
@@ -160,13 +175,26 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
           java-version: '8.0.302'
 
+      - name: Setup gradle properties
+        run: |
+          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
+
       - name: Get Semver
         id: semver
         if: ${{ startsWith(github.ref, 'refs/heads/release/v') }}
@@ -205,11 +233,6 @@ jobs:
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Setup gradle properties
-        run: |
-          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
-
       - name: Create Dockerfile and context
         uses: burrunan/gradle-cache-action@v1
         with:
@@ -256,14 +279,26 @@ jobs:
             org.opencontainers.image.description=Deephaven web
             org.opencontainers.image.licenses=Deephaven Community License Agreement 1.0
 
-      # note: this uses Zulu and not AdoptOpenJDK or other. should make sure we build and test on the same one...
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
           java-version: '8.0.302'
 
+      - name: Setup gradle properties
+        run: |
+          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
+
       - name: Set up Docker Buildx
         id: buildx
         uses: docker/setup-buildx-action@v1.3.0
@@ -279,11 +314,6 @@ jobs:
           username: ${{ github.actor }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Setup gradle properties
-        run: |
-          cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
-
       - name: Create Dockerfile and context
         uses: burrunan/gradle-cache-action@v1
         with:

diff --git a/.github/workflows/check-ci.yml b/.github/workflows/check-ci.yml
@@ -16,9 +16,16 @@ jobs:
       - name: Check gitignore rules
         run: .github/scripts/check-gitignore-rules.sh
 
-      # note: this uses Zulu and not AdoptOpenJDK or other. should make sure we build and test on the same one...
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
@@ -27,7 +34,7 @@ jobs:
       - name: Setup gradle properties
         run: |
           cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
 
       - name: Check
         uses: burrunan/gradle-cache-action@v1

diff --git a/.github/workflows/docs-ci.yml b/.github/workflows/docs-ci.yml
@@ -13,24 +13,32 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Setup JDK 8
-        id: setup-java-8
+      - name: Setup JDK 15
+        id: setup-java-15
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
-          java-version: '8.0.302'
+          java-version: '15.0.4'
 
-      - name: Setup JDK 15
-        id: setup-java-15
+      - name: Setup JDK 17
+        id: setup-java-17
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
-          java-version: '15.0.4'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '8.0.302'
 
       - name: Setup gradle properties
         run: |
           cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-15.outputs.path }}" >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-15.outputs.path }},${{ steps.setup-java-17.outputs.path }}" >> gradle.properties
 
       - name: All Javadoc
         uses: burrunan/gradle-cache-action@v1
@@ -64,23 +72,32 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Setup JDK 8
-        id: setup-java-8
-        uses: actions/setup-java@v2.2.0
-        with:
-          distribution: 'zulu'
-          java-version: '8.0.302'
       - name: Setup JDK 15
         id: setup-java-15
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
           java-version: '15.0.4'
 
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '8.0.302'
+
       - name: Setup gradle properties
         run: |
           cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-15.outputs.path }}" >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-15.outputs.path }},${{ steps.setup-java-17.outputs.path }}" >> gradle.properties
 
       - name: Generate Python Docs
         uses: burrunan/gradle-cache-action@v1

diff --git a/.github/workflows/long-check-ci.yml b/.github/workflows/long-check-ci.yml
@@ -13,9 +13,16 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      # note: this uses Zulu and not AdoptOpenJDK or other. should make sure we build and test on the same one...
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
@@ -24,7 +31,7 @@ jobs:
       - name: Setup gradle properties
         run: |
           cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
 
       - name: Check
         uses: burrunan/gradle-cache-action@v1

diff --git a/.github/workflows/nightly-benchmarks.yml b/.github/workflows/nightly-benchmarks.yml
@@ -14,9 +14,16 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v2
 
-      # note: this uses Zulu and not AdoptOpenJDK or other. should make sure we build and test on the same one...
-      - name: Setup JDK
-        id: setup-java
+      - name: Setup JDK 17
+        id: setup-java-17
+        uses: actions/setup-java@v2.2.0
+        with:
+          distribution: 'zulu'
+          java-version: '17.0.0'
+
+      # Note: setting up JDK 8 last, the current version of gradle can't be run off of 17 yet.
+      - name: Setup JDK 8
+        id: setup-java-8
         uses: actions/setup-java@v2.2.0
         with:
           distribution: 'zulu'
@@ -25,7 +32,7 @@ jobs:
       - name: Setup gradle properties
         run: |
           cat .github/env/${{ runner.os }}/gradle.properties >> gradle.properties
-          echo "org.gradle.java.installations.paths=${{ steps.setup-java.outputs.path }}" >> gradle.properties
+          echo "org.gradle.java.installations.paths=${{ steps.setup-java-8.outputs.path }},${{ steps.setup-java-17.outputs.path }}," >> gradle.properties
 
       - name: Build
         uses: burrunan/gradle-cache-action@v1