-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Backport] [2.x] Update bundled JDK to JDK-21.0.1 (#10576) #11003
[Backport] [2.x] Update bundled JDK to JDK-21.0.1 (#10576) #11003
Conversation
Gradle Check (Jenkins) Run Completed with:
|
9423fe1
to
3d7aa59
Compare
Gradle Check (Jenkins) Run Completed with:
|
Compatibility status:Checks if related components are compatible with change cef267e Incompatible componentsIncompatible components: [https://github.com/opensearch-project/performance-analyzer.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git] |
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## 2.x #11003 +/- ##
=========================================
Coverage 70.90% 70.90%
- Complexity 58928 58957 +29
=========================================
Files 4862 4862
Lines 278394 278401 +7
Branches 40810 40812 +2
=========================================
+ Hits 197396 197408 +12
- Misses 64189 64280 +91
+ Partials 16809 16713 -96
|
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.
@reta I'm going to approve but as there might be downstream impact, please merge when you think it the best time
I don't know what the implications of this change are. |
At a high level, this might cause Gradle compatibility issues across multiple repos so we need to start a campaign soon. |
Thanks @bbarani, for Gradle specifically - no issues and no changes, this is only runtime change (which is as important as Gradle): we would need to make sure that all plugins work under JDK-21 as a runtime.
Thanks @dblock , explained above |
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.
I don't have any strong opinions of when to merge this, your call @reta, PR approved.
IMO downstream plugin pain is not solvable without the kind of decoupling introduced by extensions
Sure, but this change needs awareness and alignment that it is OK to target 2.12 |
Signed-off-by: Andriy Redko <[email protected]> (cherry picked from commit 0d7d1e9) Signed-off-by: Andriy Redko <[email protected]>
…rch-project#5279) This switch is removed in JDK version 17. https://openjdk.org/jeps/403 Signed-off-by: Rabi Panda <[email protected]>
…nsearch-project#5205) Signed-off-by: Rabi Panda <[email protected]>
Signed-off-by: Andriy Redko <[email protected]>
fa560e8
to
cef267e
Compare
Gradle Check (Jenkins) Run Completed with:
|
Seeing no objections, merging JDK-21 into 2.x branch |
Backport of #10576 to
2.x