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 1.x] Fix dependencies (#4963) #4981

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 vachshah@amazon.com

  • Upgrading indirect snakeyaml dependency

Signed-off-by: Vacha Shah vachshah@amazon.com

  • Update CHANGELOG

Signed-off-by: Vacha Shah vachshah@amazon.com

  • Using snakeyaml version from version file

Signed-off-by: Vacha Shah vachshah@amazon.com

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

Signed-off-by: Vacha Shah vachshah@amazon.com

  • Extracting kotlin version

Signed-off-by: Vacha Shah vachshah@amazon.com

Signed-off-by: Vacha Shah vachshah@amazon.com

Description

Backport #4963 to 1.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 <vachshah@amazon.com>

* Upgrading indirect snakeyaml dependency

Signed-off-by: Vacha Shah <vachshah@amazon.com>

* Update CHANGELOG

Signed-off-by: Vacha Shah <vachshah@amazon.com>

* Using snakeyaml version from version file

Signed-off-by: Vacha Shah <vachshah@amazon.com>

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

Signed-off-by: Vacha Shah <vachshah@amazon.com>

* Extracting kotlin version

Signed-off-by: Vacha Shah <vachshah@amazon.com>

Signed-off-by: Vacha Shah <vachshah@amazon.com>
@VachaShah VachaShah requested review from a team and reta as code owners October 30, 2022 22:32
Signed-off-by: Vacha Shah <vachshah@amazon.com>
@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 #4981 (cad95c1) into 1.x (6d5b6fd) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                1.x    #4981      +/-   ##
============================================
+ Coverage     77.77%   77.87%   +0.10%     
- Complexity    63391    63440      +49     
============================================
  Files          4453     4453              
  Lines        275011   275011              
  Branches      41172    41172              
============================================
+ Hits         213891   214169     +278     
+ Misses        44189    43920     -269     
+ Partials      16931    16922       -9     
Impacted Files Coverage Δ
...h/index/engine/EngineCreationFailureException.java 50.00% <0.00%> (-50.00%) ⬇️
...n/indices/upgrade/post/UpgradeSettingsRequest.java 30.00% <0.00%> (-45.00%) ⬇️
...regations/metrics/AbstractHyperLogLogPlusPlus.java 51.72% <0.00%> (-37.94%) ⬇️
...r/src/main/java/org/opensearch/http/HttpUtils.java 33.33% <0.00%> (-33.34%) ⬇️
.../java/org/opensearch/index/reindex/RemoteInfo.java 49.55% <0.00%> (-26.55%) ⬇️
...nsearch/test/engine/ThrowingLeafReaderWrapper.java 40.00% <0.00%> (-20.00%) ⬇️
...positories/get/TransportGetRepositoriesAction.java 68.75% <0.00%> (-18.75%) ⬇️
...ations/bucket/terms/heuristic/PercentageScore.java 34.78% <0.00%> (-17.40%) ⬇️
...rc/main/java/org/opensearch/ingest/IngestInfo.java 51.72% <0.00%> (-17.25%) ⬇️
.../main/java/org/opensearch/action/StepListener.java 83.33% <0.00%> (-16.67%) ⬇️
... and 458 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 e1c682a into opensearch-project:1.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