You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2017/05/19 18:09:39 UTC

hbase git commit: HBASE-18071 Fix flaky test TestStochasticLoadBalancer#testBalanceCluster

Repository: hbase
Updated Branches:
  refs/heads/master 5c728b0f0 -> 8b70d043e


HBASE-18071 Fix flaky test TestStochasticLoadBalancer#testBalanceCluster

Test was failing on clusters with large number of servers or regions. Using commonly using config settings like some other tests seems to work.

Signed-off-by: Michael Stack <st...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/8b70d043
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/8b70d043
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/8b70d043

Branch: refs/heads/master
Commit: 8b70d043e48c2dd5fdf304ee6b1212c3d8006de7
Parents: 5c728b0
Author: Umesh Agashe <ua...@cloudera.com>
Authored: Fri May 19 10:02:45 2017 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Fri May 19 11:09:28 2017 -0700

----------------------------------------------------------------------
 .../hbase/master/balancer/TestStochasticLoadBalancer.java       | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/8b70d043/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java
index 9d193d2..a20925b 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java
@@ -118,7 +118,10 @@ public class TestStochasticLoadBalancer extends BalancerTestBase {
    */
   @Test
   public void testBalanceCluster() throws Exception {
-
+    conf.setLong(StochasticLoadBalancer.MAX_STEPS_KEY, 2000000L);
+    conf.setLong("hbase.master.balancer.stochastic.maxRunningTime", 90 * 1000); // 90 sec
+    conf.setFloat("hbase.master.balancer.stochastic.maxMovePercent", 1.0f);
+    loadBalancer.setConf(conf);
     for (int[] mockCluster : clusterStateMocks) {
       Map<ServerName, List<HRegionInfo>> servers = mockClusterServers(mockCluster);
       List<ServerAndLoad> list = convertToList(servers);