Skip to content

Remove JDK 11 from root pom#18751

Merged
jtuglu1 merged 1 commit intoapache:masterfrom
jtuglu1:remove-jdk-11
Nov 19, 2025
Merged

Remove JDK 11 from root pom#18751
jtuglu1 merged 1 commit intoapache:masterfrom
jtuglu1:remove-jdk-11

Conversation

@jtuglu1
Copy link
Copy Markdown
Contributor

@jtuglu1 jtuglu1 commented Nov 18, 2025

Description

V35 removed support for JDK 11. This ensures using proper maven compiler version.

Release note

Switch to using JDK 17 Maven compiler version


This PR has:

  • been self-reviewed.
  • added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

Copy link
Copy Markdown
Contributor

@cryptoe cryptoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 after CI passes.

@jtuglu1
Copy link
Copy Markdown
Contributor Author

jtuglu1 commented Nov 18, 2025

Looks like this compiler change has caused some reproducible test failures.

Edit: I think the errors are actually introduced from this commit. Applying Gian's patch.

@jtuglu1 jtuglu1 merged commit cf8f995 into apache:master Nov 19, 2025
101 of 104 checks passed
cryptoe added a commit to cryptoe/druid that referenced this pull request Nov 19, 2025
cryptoe added a commit that referenced this pull request Nov 20, 2025
@kgyrtkirk kgyrtkirk added this to the 36.0.0 milestone Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants