From 26d4f996fea402e595c91a6ef1a64fb991b99005 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Isma=C3=ABl=20Mej=C3=ADa?= Date: Sat, 23 May 2026 11:49:58 +0200 Subject: [PATCH] Simplify CI: collapse codec matrix and add zstd coverage Remove the codec matrix dimension from CI Hadoop 3 workflow. The codec split only parameterized a single integration test (FileEncodingsIT) while duplicating the entire build+test suite, wasting ~46% of billable CI minutes for <1 min of parallelized test execution. Now all codecs (uncompressed, brotli, gzip, snappy, zstd) run in a single verify step per JDK version. Also adds zstd to the test matrix. --- .github/workflows/ci-hadoop3.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci-hadoop3.yml b/.github/workflows/ci-hadoop3.yml index 7ce1820cd2..6cb39a4e3e 100644 --- a/.github/workflows/ci-hadoop3.yml +++ b/.github/workflows/ci-hadoop3.yml @@ -27,8 +27,7 @@ jobs: fail-fast: false matrix: java: [ { setup: '11', maven: '11' }, { setup: '17', maven: '17' } ] - codes: [ 'uncompressed,brotli', 'gzip,snappy' ] - name: Build Parquet with JDK ${{ matrix.java.setup }} and ${{ matrix.codes }} + name: Build Parquet with JDK ${{ matrix.java.setup }} steps: - uses: actions/checkout@master @@ -49,7 +48,7 @@ jobs: ./mvnw install --batch-mode -DskipTests=true -Dmaven.javadoc.skip=true -Dsource.skip=true -Djava.version=${{ matrix.java.maven }} - name: verify env: - TEST_CODECS: ${{ matrix.codes }} + TEST_CODECS: 'uncompressed,brotli,gzip,snappy,zstd' JAVA_VERSION: ${{ matrix.java.setup }} run: | EXTRA_JAVA_TEST_ARGS=$(./mvnw help:evaluate -Dexpression=extraJavaTestArgs -q -DforceStdout)