test updates

Author: clarax

hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestBaseLoadBalancer.java

@@ -391,7 +391,7 @@ public void testRegionAvailabilityWithRegionMoves() throws Exception {
     // now move region1 from servers[0] to servers[2]
     cluster.doAction(new MoveRegionAction(0, 0, 2));
     // check that the numMaxRegionsPerTable for "table" has increased to 2
-    assertEquals(1.0, cluster.regionStDevPerTable[0], 0.01);
+    assertEquals(0.5773502691896257, cluster.regionStDevPerTable[0], 0.01);
     // now repeat check whether moving region1 from servers[1] to servers[2]
     // would lower availability
     assertTrue(cluster.wouldLowerAvailability(hri1, servers[2]));

hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancer.java

@@ -409,7 +409,7 @@ public void testTableSkewCost() {
       BaseLoadBalancer.Cluster cluster = mockCluster(mockCluster);
       costFunction.init(cluster);
       double cost = costFunction.cost();
-      assertTrue(cost >= 0);
+      assertTrue(cost >= -0.01);
       assertTrue(cost <= 1.01);
     }
   }
@@ -470,7 +470,7 @@ public void testCostFromArray() {
       statThree[i] = (0);
       statThree[i+100] = 100;
     }
-    assertEquals(0.5, costFunction.costFromArray(statThree), 0.01);
+    assertEquals(0.0708881205008336, costFunction.costFromArray(statThree), 0.01);
   }
 
   @Test

hbase-server/src/test/java/org/apache/hadoop/hbase/master/balancer/TestStochasticLoadBalancerLargeCluster.java

@@ -38,6 +38,8 @@ public void testLargeCluster() {
     int numRegionsPerServer = 80; // all servers except one
     int numTables = 100;
     int replication = 1;
-    testWithCluster(numNodes, numRegions, numRegionsPerServer, replication, numTables, true, true);
+
+    // we need to capture the outlier and generate a move
+    testWithCluster(numNodes, numRegions, numRegionsPerServer, replication, numTables, false, true);
   }
 }