Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HBASE-18634 HBASE-Fix client.TestClientClusterStatus (Chia-Ping Tsai) 

This is a revert of a revert; i.e. a reapply just to fix commit message.

This reverts commit fbbae8774c.
This commit is contained in:
Michael Stack 2017-08-21 14:38:58 -07:00
parent 7a16acc881
commit 61b2508f29
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestClientClusterStatus.java
View File

@ -65,7 +65,7 @@ public class TestClientClusterStatus {
RegionServerThread rst = rsts.get(rsts.size() - 1); RegionServerThread rst = rsts.get(rsts.size() - 1);
DEAD = rst.getRegionServer(); DEAD = rst.getRegionServer();
DEAD.stop("Test dead servers status"); DEAD.stop("Test dead servers status");
while (!DEAD.isStopped()) { while (rst.isAlive()) {
Thread.sleep(500); Thread.sleep(500);
} }
} }
@ -155,9 +155,11 @@ public class TestClientClusterStatus {
Assert.assertNotNull(status); Assert.assertNotNull(status);
Assert.assertNotNull(status.getServers()); Assert.assertNotNull(status.getServers());
// exclude a dead region server // exclude a dead region server
Assert.assertEquals(SLAVES, numRs); Assert.assertEquals(SLAVES -1, numRs);
// live servers = primary master + nums of regionservers // live servers = nums of regionservers
Assert.assertEquals(status.getServers().size() + 1 /*Master*/, numRs); // By default, HMaster don't carry any regions so it won't report its load.
// Hence, it won't be in the server list.
Assert.assertEquals(status.getServers().size(), numRs);
Assert.assertTrue(status.getRegionsCount() > 0); Assert.assertTrue(status.getRegionsCount() > 0);
Assert.assertNotNull(status.getDeadServerNames()); Assert.assertNotNull(status.getDeadServerNames());
Assert.assertEquals(1, status.getDeadServersSize()); Assert.assertEquals(1, status.getDeadServersSize());