Skip to content
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

OpenJDK Update (January 2023 Patch releases) #6074

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

reta
Copy link
Collaborator

@reta reta commented Jan 30, 2023

Signed-off-by: Andriy Redko andriy.redko@aiven.io

Description

OpenJDK Update (January 2023 Patch releases)

Issues Resolved

Closes #6073

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #6074 (dea5a89) into main (d5a87af) will increase coverage by 0.07%.
The diff coverage is 71.42%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##               main    #6074      +/-   ##
============================================
+ Coverage     70.68%   70.76%   +0.07%     
- Complexity    58700    58787      +87     
============================================
  Files          4774     4775       +1     
  Lines        280903   280993      +90     
  Branches      40582    40590       +8     
============================================
+ Hits         198564   198851     +287     
+ Misses        65975    65802     -173     
+ Partials      16364    16340      -24     
Impacted Files Coverage Δ
...a/org/opensearch/gradle/test/DistroTestPlugin.java 0.00% <ø> (ø)
...in/cluster/health/ClusterHealthRequestBuilder.java 27.77% <0.00%> (-1.64%) ⬇️
...g/opensearch/cluster/coordination/Coordinator.java 75.36% <ø> (-3.13%) ⬇️
...ster/decommission/NodeDecommissionedException.java 40.00% <0.00%> (-10.00%) ⬇️
...pensearch/common/settings/IndexScopedSettings.java 100.00% <ø> (ø)
...n/cluster/health/TransportClusterHealthAction.java 45.26% <20.00%> (-0.69%) ⬇️
...earch/common/util/concurrent/AsyncIOProcessor.java 92.98% <66.66%> (-1.02%) ⬇️
...mmon/util/concurrent/BufferedAsyncIOProcessor.java 68.96% <68.96%> (ø)
...ion/admin/cluster/health/ClusterHealthRequest.java 78.51% <70.00%> (-0.77%) ⬇️
...org/opensearch/cluster/metadata/IndexMetadata.java 84.36% <76.92%> (-0.12%) ⬇️
... and 503 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@andrross
Copy link
Member

Why does the main branch not need the change to DateUtilsTest?

@reta
Copy link
Collaborator Author

reta commented Jan 30, 2023

Why does the main branch not need the change to DateUtilsTest?

Because of different JDK (and Joda) version, I will update Joda in 2.x (in separate PR, don't want to bundle it up with OpenJDK)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.geo.search.aggregations.bucket.GeoHashGridIT.testGeoShapes

@reta reta merged commit b7e605d into opensearch-project:main Jan 30, 2023
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.

OpenJDK Update (January 2023 Patch releases)
3 participants