Skip to content

Drop JDK 21 from the PR build matrix - #1937

Merged
Croway merged 1 commit into
apache:mainfrom
Croway:ci-drop-jdk21-pr-matrix
Sep 2, 2026
Merged

Drop JDK 21 from the PR build matrix#1937
Croway merged 1 commit into
apache:mainfrom
Croway:ci-drop-jdk21-pr-matrix

Conversation

@Croway

@Croway Croway commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

The PR build workflow runs the full incremental build once per JDK, so every push to a PR enqueues three jobs (17, 21, 25). With the current volume of open PRs the queue backs up and reviewers wait on checks.

JDK 17 is the minimum supported version (#1718) and JDK 25 is the latest LTS, so the two of them bound what a JDK 21 run would catch. This change drops 21 from the matrix, cutting every PR's CI cost by a third.

-        java: ['17', '21', '25']
+        java: ['17', '25']

Only pr-build-main.yml has a JDK matrix; the sync and SBOM workflows already run on a fixed JDK 17 and are unchanged. Existing open PRs keep the three-job matrix until they are rebased on main.

Claude Code on behalf of Federico Mariani

The PR build runs the full incremental build once per JDK, so every PR
enqueues three jobs. JDK 17 is the minimum supported version and JDK 25
is the latest LTS, and these two bound what JDK 21 would catch. Testing
on 17 and 25 only cuts the PR queue by a third.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@Croway
Croway requested review from apupier and oscerd September 2, 2026 13:57
@Croway
Croway merged commit 1d42cd7 into apache:main Sep 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants