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

Integrate translog cleanup with snapshot deletion and fix primary term deletion logic #15657

Merged
merged 6 commits into from
Sep 4, 2024

Fix tests

8663344
Select commit
Loading
Failed to load commit list.
Merged

Integrate translog cleanup with snapshot deletion and fix primary term deletion logic #15657

Fix tests
8663344
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Sep 4, 2024 in 1s

40.62% of diff hit (target 71.85%)

View this Pull Request on Codecov

40.62% of diff hit (target 71.85%)

Annotations

Check warning on line 120 in server/src/main/java/org/opensearch/index/store/RemoteSegmentStoreDirectoryFactory.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/store/RemoteSegmentStoreDirectoryFactory.java#L120 

Added line #L120 was not covered by tests

Check warning on line 406 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#L405-L406 

Added lines #L405 - L406 were not covered by tests

Check warning on line 436 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#L435-L436 

Added lines #L435 - L436 were not covered by tests

Check warning on line 452 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#L452 

Added line #L452 was not covered by tests

Check warning on line 456 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#L454-L456 

Added lines #L454 - L456 were not covered by tests

Check warning on line 463 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#L459-L463 

Added lines #L459 - L463 were not covered by tests

Check warning on line 465 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#L465 

Added line #L465 was not covered by tests

Check warning on line 470 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#L468-L470 

Added lines #L468 - L470 were not covered by tests

Check warning on line 476 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#L476 

Added line #L476 was not covered by tests

Check warning on line 479 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#L479 

Added line #L479 was not covered by tests

Check warning on line 484 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#L483-L484 

Added lines #L483 - L484 were not covered by tests

Check warning on line 486 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#L486 

Added line #L486 was not covered by tests

Check warning on line 489 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#L488-L489 

Added lines #L488 - L489 were not covered by tests

Check warning on line 491 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#L491 

Added line #L491 was not covered by tests

Check warning on line 496 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#L494-L496 

Added lines #L494 - L496 were not covered by tests

Check warning on line 499 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#L499 

Added line #L499 was not covered by tests

Check warning on line 502 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#L502 

Added line #L502 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#L509-L512 

Added lines #L509 - L512 were not covered by tests

Check warning on line 517 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#L516-L517 

Added lines #L516 - L517 were 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 553 in server/src/main/java/org/opensearch/index/translog/transfer/TranslogTransferManager.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/org/opensearch/index/translog/transfer/TranslogTransferManager.java#L553 

Added line #L553 was not covered by tests

Check warning on line 130 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#L130 

Added line #L130 was not covered by tests

Check warning on line 192 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#L192 

Added line #L192 was not covered by tests

Check warning on line 197 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#L194-L197 

Added lines #L194 - L197 were not covered by tests

Check warning on line 2151 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#L2151 

Added line #L2151 was not covered by tests
View more details on Codecov