Skip to content

[Backport 2.17] Add dedicated string prefix for remote index metadata and remote routing table#15740

Merged
gbbafna merged 1 commit intoopensearch-project:2.17from
soosinha:backport/backport-15575-to-2.17
Sep 6, 2024
Merged

[Backport 2.17] Add dedicated string prefix for remote index metadata and remote routing table#15740
gbbafna merged 1 commit intoopensearch-project:2.17from
soosinha:backport/backport-15575-to-2.17

Conversation

@soosinha
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown
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.

@codecov
Copy link
Copy Markdown

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
@github-actions
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 5, 2024

✅ Gradle check result for 5ce8f5e: SUCCESS

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