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] Purge remote translog basis the latest metadata for remote-backed indexes #6168

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 54ce423 from #6086.

…exes (#6086)

* Deleting remote translog considering latest remote metadata

Co-authored-by: Gaurav Bafna <gbbafna@amazon.com>
Signed-off-by: Ashish Singh <ssashish@amazon.com>
(cherry picked from commit 54ce423)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #6168 (54ce423) into 2.x (31b4eb5) will increase coverage by 0.39%.
The diff coverage is 49.59%.

❗ Current head 54ce423 differs from pull request most recent head de235fe. Consider uploading reports for the commit de235fe to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #6168      +/-   ##
============================================
+ Coverage     70.50%   70.90%   +0.39%     
+ Complexity    59052    58904     -148     
============================================
  Files          4772     4778       +6     
  Lines        282783   281149    -1634     
  Branches      41187    40622     -565     
============================================
- Hits         199380   199346      -34     
+ Misses        66710    65480    -1230     
+ Partials      16693    16323     -370     
Impacted Files Coverage Δ
...ark/time/NanoTimeVsCurrentTimeMillisBenchmark.java 0.00% <0.00%> (ø)
...nsearch/gradle/precommit/LicenseHeadersTask.groovy 0.00% <ø> (ø)
.../java/org/opensearch/gradle/JdkDownloadPlugin.java 36.26% <0.00%> (-1.10%) ⬇️
...rg/opensearch/gradle/OpenSearchTestBasePlugin.java 6.81% <0.00%> (-0.25%) ⬇️
...org/opensearch/gradle/RepositoriesSetupPlugin.java 38.63% <0.00%> (ø)
...ternal/InternalDistributionArchiveCheckPlugin.java 0.00% <0.00%> (ø)
...ternal/InternalDistributionArchiveSetupPlugin.java 0.00% <ø> (ø)
...ensearch/gradle/precommit/FilePermissionsTask.java 78.78% <ø> (ø)
...search/gradle/precommit/ForbiddenPatternsTask.java 88.40% <ø> (ø)
...earch/gradle/precommit/JarHellPrecommitPlugin.java 31.25% <0.00%> (ø)
... and 1028 more

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Gradle Check (Jenkins) Run Completed with:

@kotwanikunal
Copy link
Member

Closing in favor of #6170

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.

4 participants