This is an automated email from the ASF dual-hosted git repository.
gian pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new 2a09371cf7f Run JDK 21 tests on PRs (#18717)
2a09371cf7f is described below
commit 2a09371cf7fd767113e9cbe8fcdf0e7247a500de
Author: Abhishek Radhakrishnan <[email protected]>
AuthorDate: Fri Nov 7 14:07:11 2025 -0800
Run JDK 21 tests on PRs (#18717)
---
.github/workflows/ci.yml | 20 +++-----------------
.../druid/java/util/common/CompressionUtilsTest.java | 4 +++-
2 files changed, 6 insertions(+), 18 deletions(-)
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index f7f6c9d3c2a..bc8389dc656 100644
--- a/.github/workflows/ci.yml
+++ b/.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:
diff --git
a/processing/src/test/java/org/apache/druid/java/util/common/CompressionUtilsTest.java
b/processing/src/test/java/org/apache/druid/java/util/common/CompressionUtilsTest.java
index f02e441573a..27377a53a77 100644
---
a/processing/src/test/java/org/apache/druid/java/util/common/CompressionUtilsTest.java
+++
b/processing/src/test/java/org/apache/druid/java/util/common/CompressionUtilsTest.java
@@ -33,6 +33,7 @@ import
org.apache.commons.compress.compressors.zstandard.ZstdCompressorOutputStr
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 class CompressionUtilsTest
}
// 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);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
