Skip to content

Update bundled JDK to 21.0.12.1+1 - #22809

Open
reta wants to merge 1 commit into
opensearch-project:2.19from
reta:jdk.21.0.12.1+1
Open

Update bundled JDK to 21.0.12.1+1#22809
reta wants to merge 1 commit into
opensearch-project:2.19from
reta:jdk.21.0.12.1+1

Conversation

@reta

@reta reta commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Description

Update bundled JDK to 21.0.12.1+1 (new JDK CPU release cadence).
See please https://www.oracle.com/security-alerts/cspuaug2026.html#AppendixJAVA

Related Issues

N/A

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta
reta requested a review from a team as a code owner August 21, 2026 18:00
@reta reta added Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. labels Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit a9e9533)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

Regex Capture Group Shift

The updated VERSION_PATTERN regex changed the minor version group from (\\.\\d+\\.\\d+)? to (\\.\\d+\\.\\d+(?:\\.\\d+)?)?. While this now matches versions like 21.0.12.1+1, callers relying on capture group indices to extract the build/hash portions still work because the added (?:...) is non-capturing. However, verify that any code parsing the version string (e.g., extracting the fourth component .1) handles the optional fourth segment correctly, as it is now bundled inside group 2 rather than being a separate group. Downstream consumers expecting a strict three-part version may misparse the new format.

private static final Pattern VERSION_PATTERN = Pattern.compile(
    "(\\d+)(\\.\\d+\\.\\d+(?:\\.\\d+)?)?\\+(\\d+(?:\\.\\d+)?)(@([a-f0-9]{32}))?"
);

Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta
reta force-pushed the jdk.21.0.12.1+1 branch from 3b6f307 to a9e9533 Compare August 21, 2026 18:02
@github-actions

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit a9e9533

@github-actions

Copy link
Copy Markdown
Contributor

❌ Gradle check result for a9e9533: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant