Run JDK 21 tests on PRs (#18717)

Author: abhishekrb19

.github/workflows/ci.yml

@@ -24,13 +24,14 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        jdk: [ "17", "21" ]
         pattern: [ "C*", "H*,U*,V*", "N*,Q*,S*", "B*,O*,R*", "G*,J*,K*", "F*,L*,M*", "A*,D*,I*,X*,Y*,Z*", "E*,P*,T*,W*"]
     uses: ./.github/workflows/worker.yml
     with:
       script: .github/scripts/run_unit-tests -Dtest=!QTest,'${{ matrix.pattern }}' -Dmaven.test.failure.ignore=true
-      jdk: 17
       artifact_prefix: "unit-test-reports"
-      key: "test-jdk17-[${{ matrix.pattern }}]"
+      jdk: ${{ matrix.jdk }}
+      key: "test-jdk${{ matrix.jdk }}-[${{ matrix.pattern }}]"
 
   validate-dist:
     uses: ./.github/workflows/worker.yml
@@ -71,21 +72,6 @@ jobs:
       artifacts_to_download: "unit-test-reports-*"
       key: "coverage-jacoco"
 
-  run-unit-tests-all-jdk:
-    needs: test-report
-    name: "unit tests(all jdk)"
-    if: ${{ !cancelled() && ( contains( github.event.pull_request.labels.*.name, 'tests:all-jdk') || github.event_name == 'push' ) }}
-    strategy:
-      fail-fast: false
-      matrix:
-        jdk: [ "21" ]
-        pattern: [ "C*", "H*,U*,V*", "N*,Q*,S*", "B*,O*,R*", "G*,J*,K*", "F*,L*,M*", "A*,D*,I*,X*,Y*,Z*", "E*,P*,T*,W*"]
-    uses: ./.github/workflows/worker.yml
-    with:
-      script: .github/scripts/run_unit-tests -Dtest=!QTest,'${{ matrix.pattern }}' -fae
-      jdk: ${{ matrix.jdk }}
-      key: "test-jdk${{ matrix.jdk }}-[${{ matrix.pattern }}]"
-
   # this will be running in parallel with the ITs later; but until that migration happens - run them in parallel with normal tests
   run-qtest:
     strategy:

processing/src/test/java/org/apache/druid/java/util/common/CompressionUtilsTest.java

@@ -33,6 +33,7 @@
 import org.apache.druid.utils.CompressionUtils;
 import org.junit.Assert;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
@@ -551,8 +552,9 @@ public void testZeroRemainingInputStream() throws IOException
   }
 
   // If this ever passes, er... fails to fail... then the bug is fixed
-  @Test(expected = AssertionError.class)
   // http://bugs.java.com/bugdatabase/view_bug.do?bug_id=7036144
+  @Test(expected = AssertionError.class)
+  @Ignore("This test fails in JDK 21, looks like the bug was fixed in 21.0.9 at least.")
   public void testGunzipBug() throws IOException
   {
     final ByteArrayOutputStream tripleGzByteStream = new ByteArrayOutputStream(GZ_BYTES.length * 3);