-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Drop JDK 11 support #16443
Drop JDK 11 support #16443
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check Java version references in
Co-authored-by: João Duarte <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor tweaks, otherwise LGTM
Co-authored-by: João Duarte <[email protected]>
Quality Gate passedIssues Measures |
💚 Build Succeeded
History
cc @andsel |
Release notes
Remove support for JDK 11
What does this PR do?
Updates the
jvm.options
andJvmOptionsParser
to remove support for JDK 11. If the options parser identifies that the running JVM is less than 17, it refuses to start.Why is it important/What is the impact to the user?
If a user runs Logstash with a hosted JDK and not the one bundled with Logstash distribution, like setting a specific
LS_JAVA_HOME
, which is minor than JDK 17 then Logstash refuses to start. Has to provide at least a JDK 17 or unset theLS_JAVA_HOME
and let Logstash uses the bundled JDK.Checklist
[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have added tests that prove my fix is effective or that my feature worksAuthor's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs