This is an automated email from the ASF dual-hosted git repository.

jxue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git


The following commit(s) were added to refs/heads/master by this push:
     new 525498257 Add retry for the test
525498257 is described below

commit 525498257341a27a68cc571456b8b7c9ee7e0556
Author: Junkai Xue <[email protected]>
AuthorDate: Thu Apr 9 22:07:33 2026 -0700

    Add retry for the test
---
 .../helix/integration/rebalancer/TestInstanceOperation.java       | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git 
a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
 
b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
index 13902d64c..157cb5fb7 100644
--- 
a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
+++ 
b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestInstanceOperation.java
@@ -1069,9 +1069,11 @@ public class TestInstanceOperation extends ZkTestBase {
         VERIFICATION_TIMEOUT);
     Assert.assertTrue(swapValid, "Swap validation failed");
 
-    // Verify evacuation finished
-    boolean evacFinished = _gSetupTool.getClusterManagementTool()
-        .isEvacuateFinished(CLUSTER_NAME, swapOutName);
+    // Verify evacuation finished - use verifier to handle timing variations
+    boolean evacFinished = TestHelper.verify(() ->
+        _gSetupTool.getClusterManagementTool()
+            .isEvacuateFinished(CLUSTER_NAME, swapOutName),
+        VERIFICATION_TIMEOUT);
     LOG.info("isEvacuateFinished: {}", evacFinished);
     Assert.assertTrue(evacFinished, "Evacuation should be finished");
 

Reply via email to