-
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
Mark Java 21 default in docs. #17046
base: main
Are you sure you want to change the base?
Conversation
Warning It looks like this PR modifies one or more |
📃 DOCS PREVIEW ✨ https://logstash_bk_17046.docs-preview.app.elstc.co/diff |
The asciidoc for |
Related: #16608 |
Warning It looks like this PR modifies one or more |
📃 DOCS PREVIEW ✨ https://logstash_bk_17046.docs-preview.app.elstc.co/diff |
This pull request does not have a backport label. Could you fix it @mashhurs? 🙏
|
|
This pull request is now in conflicts. Could you fix it @mashhurs? 🙏
|
Release notes
[rn:skip]
What does this PR do?
Updates the JVM doc saying Java 17 is default. Unfortunately we changed it to Java 21 a way back.
Why is it important/What is the impact to the user?
Deliver accurate message about JDK
Checklist
[ ] My code follows the style guidelines of this project[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have made corresponding change to the default configuration files (and/or docker env variables)[ ] 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