kgusakov commented on code in PR #5064:
URL: https://github.com/apache/ignite-3/pull/5064#discussion_r1927004896
##########
modules/table/src/integrationTest/java/org/apache/ignite/internal/table/distributed/disaster/AbstractHighAvailablePartitionsRecoveryTest.java:
##########
@@ -132,10 +142,87 @@ private void
waitAndAssertStableAssignmentsOfPartitionEqualTo(
.collect(Collectors.toUnmodifiableSet()));
}
+ /**
+ * Wait for the 2 facts simultaneously.
+ *
+ * <ul>
+ * <li>All planned rebalances have finished (pending and planned keys
is empty).</li>
+ * <li>Stable assignments is equal to expected</li>
+ * </ul>
+ *
+ * @param gatewayNode Node for communication with cluster and components.
+ * @param tableName Table name.
+ * @param partitionIds Set of target partition ids to check.
+ * @param nodes Expected set of nodes in stable assignments.
+ */
+ final void
waitThatAllRebalancesHaveFinishedAndStableAssignmentsEqualsToExpected(
+ IgniteImpl gatewayNode, String tableName, Set<Integer>
partitionIds, Set<String> nodes) {
+ partitionIds.forEach(p -> {
+ try {
+
waitThatAllRebalancesHaveFinishedAndStableAssignmentsEqualsToExpected(gatewayNode,
tableName, p, nodes);
+ } catch (InterruptedException e) {
+ throw new RuntimeException(e);
+ }
+ });
+ }
+
+ /**
+ * The same as the previous one, but for concrete partition.
Review Comment:
I can't ( The link + ** is longer than 144 symbols. I tried when was making
these changes)
<img width="1559" alt="Screenshot 2025-01-23 at 16 45 42"
src="https://github.com/user-attachments/assets/bbf2155c-0245-412c-bf60-e7899574afab";
/>
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
