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

[Backport 2.x] Fix dependencies (#4963) #4982

Merged
merged 2 commits into from
Oct 31, 2022
Merged

Conversation

VachaShah
Copy link
Collaborator

  • Upgrading kotlin-stdlib and excluding jetty-http

Signed-off-by: Vacha Shah [email protected]

  • Upgrading indirect snakeyaml dependency

Signed-off-by: Vacha Shah [email protected]

  • Update CHANGELOG

Signed-off-by: Vacha Shah [email protected]

  • Using snakeyaml version from version file

Signed-off-by: Vacha Shah [email protected]

  • Upgrading jetty-http instead of excluding since it is used

Signed-off-by: Vacha Shah [email protected]

  • Extracting kotlin version

Signed-off-by: Vacha Shah [email protected]

Signed-off-by: Vacha Shah [email protected]

Description

Backport #4963 to 2.x

Issues Resolved

[List any issues this PR will resolve]

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.

* Upgrading kotlin-stdlib and excluding jetty-http

Signed-off-by: Vacha Shah <[email protected]>

* Upgrading indirect snakeyaml dependency

Signed-off-by: Vacha Shah <[email protected]>

* Update CHANGELOG

Signed-off-by: Vacha Shah <[email protected]>

* Using snakeyaml version from version file

Signed-off-by: Vacha Shah <[email protected]>

* Upgrading jetty-http instead of excluding since it is used

Signed-off-by: Vacha Shah <[email protected]>

* Extracting kotlin version

Signed-off-by: Vacha Shah <[email protected]>

Signed-off-by: Vacha Shah <[email protected]>
@VachaShah VachaShah requested review from a team and reta as code owners October 30, 2022 22:41
Signed-off-by: Vacha Shah <[email protected]>
@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 #4982 (d624c5c) into 2.x (213b966) will decrease coverage by 0.01%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##                2.x    #4982      +/-   ##
============================================
- Coverage     70.70%   70.68%   -0.02%     
- Complexity    57627    57666      +39     
============================================
  Files          4620     4620              
  Lines        276210   276220      +10     
  Branches      40421    40422       +1     
============================================
- Hits         195295   195251      -44     
- Misses        64555    64721     +166     
+ Partials      16360    16248     -112     
Impacted Files Coverage Δ
...rg/opensearch/common/settings/ClusterSettings.java 91.89% <ø> (ø)
...va/org/opensearch/indices/ShardLimitValidator.java 96.36% <83.33%> (-3.64%) ⬇️
...adonly/AddIndexBlockClusterStateUpdateRequest.java 0.00% <0.00%> (-75.00%) ⬇️
...readonly/TransportVerifyShardIndexBlockAction.java 9.75% <0.00%> (-58.54%) ⬇️
...ch/transport/ReceiveTimeoutTransportException.java 50.00% <0.00%> (-50.00%) ⬇️
...cluster/coordination/PendingClusterStateStats.java 20.00% <0.00%> (-48.00%) ⬇️
...indices/readonly/TransportAddIndexBlockAction.java 20.68% <0.00%> (-37.94%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
...ations/bucket/terms/heuristic/PercentageScore.java 17.39% <0.00%> (-34.79%) ⬇️
...ensearch/client/indices/DetailAnalyzeResponse.java 20.54% <0.00%> (-34.25%) ⬇️
... and 489 more

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

@reta reta merged commit 26abe78 into opensearch-project:2.x Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants