Skip to content

Conversation

@harishbhakuni
Copy link
Contributor

@harishbhakuni harishbhakuni commented Jun 15, 2023

… for remote store interoperability.

Description

This is a Backport request for Change #7118 .
To Avoid BWC test failures in main branch, kept version checks against Version.V_3_0_0.
now as part of this PR, backporting changes to 2.x branch and adding changes to keep version checks against desired version (Version.V_2_9_0).

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:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

… for remote store interoperability.

Signed-off-by: Harish Bhakuni <hbhakuni@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString

@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #8071 (b404fe7) into 2.x (d868857) will increase coverage by 0.09%.
The diff coverage is 73.48%.

@@             Coverage Diff              @@
##                2.x    #8071      +/-   ##
============================================
+ Coverage     70.63%   70.73%   +0.09%     
- Complexity    56741    56878     +137     
============================================
  Files          4701     4702       +1     
  Lines        268712   268993     +281     
  Branches      39724    39768      +44     
============================================
+ Hits         189798   190263     +465     
+ Misses        62641    62458     -183     
+ Partials      16273    16272       -1     
Impacted Files Coverage Δ
.../org/opensearch/repositories/FilterRepository.java 6.38% <0.00%> (-0.29%) ⬇️
...n/java/org/opensearch/repositories/Repository.java 50.00% <0.00%> (-16.67%) ⬇️
...g/opensearch/repositories/RepositoriesService.java 73.31% <50.00%> (+0.89%) ⬆️
...ava/org/opensearch/snapshots/SnapshotsService.java 43.36% <60.00%> (-1.07%) ⬇️
...in/java/org/opensearch/snapshots/SnapshotInfo.java 78.01% <62.50%> (-0.18%) ⬇️
...rg/opensearch/snapshots/SnapshotShardsService.java 63.46% <62.50%> (-7.72%) ⬇️
...in/java/org/opensearch/index/shard/IndexShard.java 70.64% <66.66%> (+1.15%) ⬆️
...ch/repositories/blobstore/BlobStoreRepository.java 62.71% <67.39%> (+0.39%) ⬆️
...blobstore/RemoteStoreShardShallowCopySnapshot.java 83.67% <83.67%> (ø)
...va/org/opensearch/cluster/SnapshotsInProgress.java 68.68% <90.90%> (-8.23%) ⬇️

... and 484 files with indirect coverage changes

@gbbafna gbbafna merged commit b642e69 into opensearch-project:2.x Jun 15, 2023
gaiksaya pushed a commit to gaiksaya/OpenSearch that referenced this pull request Jun 26, 2023
… for remote store interoperability. (opensearch-project#8071)

Signed-off-by: Harish Bhakuni <hbhakuni@amazon.com>
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.

2 participants