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.17] Add dedicated string prefix for remote index metadata and remote routing table #15740

Merged

Conversation

soosinha
Copy link
Member

@soosinha soosinha commented Sep 5, 2024

Description

Backport 1935650 from #15575

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

@soosinha soosinha changed the title Add dedicated string prefix for remote index metadata and remote routing table [Backport 2.17] Add dedicated string prefix for remote index metadata and remote routing table Sep 5, 2024
Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Gradle check result for 75b9bc9: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@soosinha soosinha force-pushed the backport/backport-15575-to-2.17 branch from 75b9bc9 to 92c8301 Compare September 5, 2024 09:12
Copy link
Contributor

github-actions bot commented Sep 5, 2024

❕ Gradle check result for 92c8301: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Copy link

codecov bot commented Sep 5, 2024

Codecov Report

Attention: Patch coverage is 97.22222% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.64%. Comparing base (75fb5bc) to head (5ce8f5e).
Report is 16 commits behind head on 2.17.

Files with missing lines Patch % Lines
...arch/gateway/remote/model/RemoteIndexMetadata.java 91.66% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               2.17   #15740      +/-   ##
============================================
- Coverage     71.66%   71.64%   -0.03%     
- Complexity    64317    64398      +81     
============================================
  Files          5262     5263       +1     
  Lines        301148   301211      +63     
  Branches      43841    43847       +6     
============================================
- Hits         215829   215800      -29     
- Misses        67259    67441     +182     
+ Partials      18060    17970      -90     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@soosinha soosinha force-pushed the backport/backport-15575-to-2.17 branch from 92c8301 to d9b26e0 Compare September 5, 2024 15:48
Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Gradle check result for d9b26e0: TIMEOUT

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

…ing table (opensearch-project#15575)

* Hashed prefix for index metadata

Signed-off-by: Sooraj Sinha <soosinha@amazon.com>
(cherry picked from commit 1935650)
@soosinha soosinha force-pushed the backport/backport-15575-to-2.17 branch from d9b26e0 to 5ce8f5e Compare September 5, 2024 17:53
Copy link
Contributor

github-actions bot commented Sep 5, 2024

✅ Gradle check result for 5ce8f5e: SUCCESS

@gbbafna gbbafna merged commit 64e503d into opensearch-project:2.17 Sep 6, 2024
35 checks passed
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.

2 participants