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 2020/12/15 17:52:57 UTC

[hbase] branch branch-2.4 updated: HBASE-25389 [Flakey Tests] branch-2 TestMetaShutdownHandler (#2773)

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

stack pushed a commit to branch branch-2.4
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2.4 by this push:
     new bdc864f  HBASE-25389 [Flakey Tests] branch-2 TestMetaShutdownHandler (#2773)
bdc864f is described below

commit bdc864fb69e543530943721f39195130fcf92dd4
Author: Michael Stack <sa...@users.noreply.github.com>
AuthorDate: Tue Dec 15 09:49:16 2020 -0800

    HBASE-25389 [Flakey Tests] branch-2 TestMetaShutdownHandler (#2773)
    
    Signed-off-by: Bharath Vissapragada <bh...@apache.org>
---
 .../java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java   | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java
index ea532da..d07654c 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java
@@ -32,6 +32,7 @@ import org.apache.hadoop.hbase.ServerName;
 import org.apache.hadoop.hbase.StartMiniClusterOption;
 import org.apache.hadoop.hbase.Waiter;
 import org.apache.hadoop.hbase.master.assignment.RegionStates;
+import org.apache.hadoop.hbase.regionserver.HRegionServer;
 import org.apache.hadoop.hbase.testclassification.MediumTests;
 import org.apache.hadoop.hbase.util.Bytes;
 import org.apache.hadoop.hbase.zookeeper.MetaTableLocator;
@@ -101,6 +102,7 @@ public class TestMetaShutdownHandler {
     RegionState metaState = MetaTableLocator.getMetaRegionState(master.getZooKeeper());
     assertEquals("Wrong state for meta!", RegionState.State.OPEN, metaState.getState());
     assertNotEquals("Meta is on master!", metaServerName, master.getServerName());
+    HRegionServer metaRegionServer = cluster.getRegionServer(metaServerName);
 
     // Delete the ephemeral node of the meta-carrying region server.
     // This is trigger the expire of this region server on the master.
@@ -112,6 +114,7 @@ public class TestMetaShutdownHandler {
     // Wait for SSH to finish
     final ServerManager serverManager = master.getServerManager();
     final ServerName priorMetaServerName = metaServerName;
+    TEST_UTIL.waitFor(60000, 100, () -> metaRegionServer.isStopped());
     TEST_UTIL.waitFor(120000, 200, new Waiter.Predicate<Exception>() {
       @Override
       public boolean evaluate() throws Exception {