sashapolo commented on code in PR #5179: URL: https://github.com/apache/ignite-3/pull/5179#discussion_r1948567237
########## modules/partition-replicator/src/integrationTest/java/org/apache/ignite/internal/partition/replicator/ItZoneDataReplicationTest.java: ########## @@ -397,6 +395,11 @@ void testDataRebalance(boolean truncateRaftLog) throws Exception { setPrimaryReplica(newNode, zonePartitionId); + // Wait for the data to appear. At the moment of writing, we don't have any partition safe time to wait for and + // the primary replica has been assigned manually, so there's no guarantee that the data has been replicated. + // Not using "assertTrue" on purpose, the next line will produce a nicer error message. + waitForCondition(() -> kvView1.getAll(null, data1.keySet()).equals(data1), 10_000L); Review Comment: Thank you, I couldn't find the relevant ticket myself -- 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