[helix] branch wagedRebalancer updated: Fix the unstable test TestZeroReplicaAvoidance. (#603)

[ji...@apache.org]

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

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


The following commit(s) were added to refs/heads/wagedRebalancer by this push:
     new 4dc35c0  Fix the unstable test TestZeroReplicaAvoidance. (#603)
4dc35c0 is described below

commit 4dc35c0414503314083c0a0de1b1869629650664
Author: Jiajun Wang <18...@users.noreply.github.com>
AuthorDate: Thu Nov 14 12:27:32 2019 -0800

    Fix the unstable test TestZeroReplicaAvoidance. (#603)
    
    Fix the unstable test TestZeroReplicaAvoidance by waiting.
    This is a temporary resolution before we fix issue #526. Marked it in the TODO comment so easier for us to remove the wait in batch later.
---
 .../apache/helix/integration/rebalancer/TestZeroReplicaAvoidance.java   | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestZeroReplicaAvoidance.java b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestZeroReplicaAvoidance.java
index bd3f2e1..1a02299 100644
--- a/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestZeroReplicaAvoidance.java
+++ b/helix-core/src/test/java/org/apache/helix/integration/rebalancer/TestZeroReplicaAvoidance.java
@@ -165,6 +165,8 @@ public class TestZeroReplicaAvoidance extends ZkTestBase
       String db = "Test-DB-" + stateModel;
       createResourceWithWagedRebalance(CLUSTER_NAME, db, stateModel, partition, replica, replica);
     }
+    // TODO remove this sleep after fix https://github.com/apache/helix/issues/526
+    Thread.sleep(1000);
     Assert.assertTrue(_clusterVerifier.verifyByPolling(50000L, 100L));
 
     _startListen = true;