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] Parallelize stale blobs deletion during snapshot delete #3990

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 1c787e8 from #3796

* Parallelize stale blobs deletion during snapshot delete

Signed-off-by: Piyush Daftary <[email protected]>

* Adding test which throws exception

Signed-off-by: Piyush Daftary <[email protected]>

* Adusting identation for spotlessJavaCheck

Signed-off-by: Piyush Daftary <[email protected]>

* Adding more description to MAX_SHARD_BLOB_DELETE_BATCH_SIZE

Signed-off-by: Piyush Daftary <[email protected]>

* Renaming max_shard_blob_delete_batch_size to max_snapshot_shard_blob_delete_batch_size

Signed-off-by: Piyush Daftary <[email protected]>
(cherry picked from commit 1c787e8)
@opensearch-trigger-bot opensearch-trigger-bot bot requested review from a team and reta as code owners July 22, 2022 20:06
@codecov-commenter
Copy link

Codecov Report

Merging #3990 (1c787e8) into 2.x (fb4f96f) will decrease coverage by 0.06%.
The diff coverage is 63.16%.

❗ Current head 1c787e8 differs from pull request most recent head d34cc53. Consider uploading reports for the commit d34cc53 to get more accurate results

@@             Coverage Diff              @@
##                2.x    #3990      +/-   ##
============================================
- Coverage     70.63%   70.57%   -0.07%     
- Complexity    56487    56797     +310     
============================================
  Files          4527     4573      +46     
  Lines        271798   273476    +1678     
  Branches      39975    40101     +126     
============================================
+ Hits         191985   193000    +1015     
- Misses        63718    64238     +520     
- Partials      16095    16238     +143     
Impacted Files Coverage Δ
...ark/time/NanoTimeVsCurrentTimeMillisBenchmark.java 0.00% <0.00%> (ø)
...nsearch/gradle/precommit/LicenseHeadersTask.groovy 0.00% <ø> (ø)
.../java/org/opensearch/gradle/JdkDownloadPlugin.java 37.36% <0.00%> (ø)
...ensearch/gradle/precommit/FilePermissionsTask.java 78.78% <ø> (ø)
...search/gradle/precommit/ForbiddenPatternsTask.java 88.40% <ø> (ø)
...g/opensearch/gradle/precommit/LoggerUsageTask.java 0.00% <ø> (ø)
...ensearch/gradle/precommit/ThirdPartyAuditTask.java 0.00% <0.00%> (ø)
...ensearch/gradle/tar/SymbolicLinkPreservingTar.java 0.00% <0.00%> (ø)
...a/org/opensearch/gradle/test/DistroTestPlugin.java 0.00% <ø> (ø)
...g/opensearch/gradle/test/rest/CopyRestApiTask.java 0.00% <ø> (ø)
... and 749 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@VachaShah VachaShah merged commit 5a7a1c2 into 2.x Jul 30, 2022
@github-actions github-actions bot deleted the backport/backport-3796-to-2.x branch July 30, 2022 01:27
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