sashapolo commented on code in PR #5588: URL: https://github.com/apache/ignite-3/pull/5588#discussion_r2039730929
########## modules/partition-replicator/src/test/java/org/apache/ignite/internal/partition/replicator/raft/snapshot/PartitionSnapshotStorageTest.java: ########## @@ -94,4 +103,56 @@ void choosesMinimalIndexFromTxStorage(@Mock PartitionMvStorageAccess partitionAc assertEquals(3L, startupSnapshotMeta.lastIncludedIndex()); assertEquals(2L, startupSnapshotMeta.lastIncludedTerm()); } + + @Test + void partitionCanBeRemovedOnlyWithoutOutgoingSnapshot(@Mock PartitionMvStorageAccess partitionAccess) throws IOException { Review Comment: Can you please clarify? I check that the storage gets eventually removed, it is required for the second scenario too -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org