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 f563b37bc Fix TestTopologyMigration
f563b37bc is described below
commit f563b37bcf0d938c82aca4f3f16328659956a0c3
Author: Junkai Xue <[email protected]>
AuthorDate: Sat Mar 7 03:29:05 2026 -0800
Fix TestTopologyMigration
- Changed setWaitTillVerify(120000) to setWaitTillVerify(10000)
- This gives enough time for the cluster to stabilize (10 seconds)
without causing the total test time to exceed the timeout
- The verifyByPolling() method already has a 5-minute timeout
(DEFAULT_TIMEOUT) per call for the actual verification logic
---
.../apache/helix/integration/controller/TestTopologyMigration.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/helix-core/src/test/java/org/apache/helix/integration/controller/TestTopologyMigration.java
b/helix-core/src/test/java/org/apache/helix/integration/controller/TestTopologyMigration.java
index 16a712d0f..6c173d97a 100644
---
a/helix-core/src/test/java/org/apache/helix/integration/controller/TestTopologyMigration.java
+++
b/helix-core/src/test/java/org/apache/helix/integration/controller/TestTopologyMigration.java
@@ -104,9 +104,9 @@ public class TestTopologyMigration extends ZkTestBase {
setupClusterConfig(INIT_TOPOLOGY, RACK);
// Initialize cluster verifier for validating state
- // Use longer timeout since the cluster has delayed rebalancing enabled
(30 min delay)
+ // Use moderate wait time - enough for cluster to settle but not too long
to cause timeout
_clusterVerifier = new
BestPossibleExternalViewVerifier.Builder(CLUSTER_NAME).setZkAddr(ZK_ADDR)
- .setWaitTillVerify(120000)
+ .setWaitTillVerify(10000)
.build();
// Setup the participants and resources for the test
