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.

Sign up for GitHub

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

Jump to bottom

Optimize remote store GC flow with pinned timestamps #15943

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
+840 −227

Fix tests

fae9f37
Select commit
Loading
Failed to load commit list.
Open

Optimize remote store GC flow with pinned timestamps #15943

Fix tests
fae9f37
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 20, 2024 in 1s

72.02% (target 70.00%)

View this Pull Request on Codecov

72.02% (target 70.00%)

Details

Codecov Report

Attention: Patch coverage is 48.14815% with 42 lines in your changes missing coverage. Please review.

Project coverage is 72.02%. Comparing base (3937ccb) to head (fae9f37).
Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
...index/translog/RemoteFsTimestampAwareTranslog.java 44.00% 24 Missing and 4 partials ⚠️
...ch/repositories/blobstore/BlobStoreRepository.java 50.00% 6 Missing ⚠️
...earch/index/store/RemoteSegmentStoreDirectory.java 50.00% 1 Missing and 2 partials ⚠️
...ex/translog/transfer/TranslogTransferMetadata.java 40.00% 3 Missing ⚠️
...ava/org/opensearch/snapshots/SnapshotsService.java 50.00% 2 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##               main   #15943      +/-   ##
============================================
+ Coverage     71.90%   72.02%   +0.12%     
- Complexity    64392    64434      +42     
============================================
  Files          5278     5280       +2     
  Lines        300877   300908      +31     
  Branches      43478    43483       +5     
============================================
+ Hits         216351   216744     +393     
+ Misses        66747    66396     -351     
+ Partials      17779    17768      -11

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

View more details on Codecov