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

Optimize remote store GC flow with pinned timestamps #15943

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

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/patch failed Sep 20, 2024 in 1s

48.14% of diff hit (target 71.90%)

View this Pull Request on Codecov

48.14% of diff hit (target 71.90%)

Annotations

Check warning on line 1008 in server/src/main/java/org/opensearch/index/store/RemoteSegmentStoreDirectory.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/store/RemoteSegmentStoreDirectory.java#L1008

Added line #L1008 was not covered by tests

Check warning on line 178 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L178

Added line #L178 was not covered by tests

Check warning on line 233 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L233

Added line #L233 was not covered by tests

Check warning on line 510 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L510

Added line #L510 was not covered by tests

Check warning on line 512 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L512

Added line #L512 was not covered by tests

Check warning on line 515 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L515

Added line #L515 was not covered by tests

Check warning on line 520 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L519-L520

Added lines #L519 - L520 were not covered by tests

Check warning on line 522 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L522

Added line #L522 was not covered by tests

Check warning on line 531 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L530-L531

Added lines #L530 - L531 were not covered by tests

Check warning on line 533 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L533

Added line #L533 was not covered by tests

Check warning on line 538 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L536-L538

Added lines #L536 - L538 were not covered by tests

Check warning on line 541 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L541

Added line #L541 was not covered by tests

Check warning on line 544 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L544

Added line #L544 was not covered by tests

Check warning on line 554 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L551-L554

Added lines #L551 - L554 were not covered by tests

Check warning on line 561 in server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/RemoteFsTimestampAwareTranslog.java#L561

Added line #L561 was not covered by tests

Check warning on line 179 in server/src/main/java/org/opensearch/index/translog/transfer/TranslogTransferMetadata.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/transfer/TranslogTransferMetadata.java#L177-L179

Added lines #L177 - L179 were not covered by tests

Check warning on line 1294 in server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java#L1294

Added line #L1294 was not covered by tests

Check warning on line 1351 in server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java#L1350-L1351

Added lines #L1350 - L1351 were not covered by tests

Check warning on line 1525 in server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java#L1525

Added line #L1525 was not covered by tests

Check warning on line 2202 in server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java#L2202

Added line #L2202 was not covered by tests

Check warning on line 2250 in server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java#L2250

Added line #L2250 was not covered by tests

Check warning on line 648 in server/src/main/java/org/opensearch/snapshots/SnapshotsService.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/snapshots/SnapshotsService.java#L647-L648

Added lines #L647 - L648 were not covered by tests