Revert "HBASE-26941 LocalHBaseCluster.waitOnRegionServer should quit while thread is interrupted (#4333)"

This reverts commit 8247b7c.

Author: Apache9

hbase-server/src/main/java/org/apache/hadoop/hbase/LocalHBaseCluster.java

@@ -300,7 +300,7 @@ public Configuration getConfiguration() {
    * Wait for the specified region server to stop. Removes this thread from list of running threads.
    * @return Name of region server that just went down.
    */
-  public String waitOnRegionServer(int serverNumber) throws InterruptedException {
+  public String waitOnRegionServer(int serverNumber) {
     JVMClusterUtil.RegionServerThread regionServerThread = this.regionThreads.get(serverNumber);
     return waitOnRegionServer(regionServerThread);
   }
@@ -309,11 +309,15 @@ public String waitOnRegionServer(int serverNumber) throws InterruptedException {
    * Wait for the specified region server to stop. Removes this thread from list of running threads.
    * @return Name of region server that just went down.
    */
-  public String waitOnRegionServer(JVMClusterUtil.RegionServerThread rst)
-    throws InterruptedException {
+  public String waitOnRegionServer(JVMClusterUtil.RegionServerThread rst) {
     while (rst.isAlive()) {
-      LOG.info("Waiting on " + rst.getRegionServer().toString());
-      rst.join();
+      try {
+        LOG.info("Waiting on " + rst.getRegionServer().toString());
+        rst.join();
+      } catch (InterruptedException e) {
+        LOG.error("Interrupted while waiting for {} to finish. Retrying join", rst.getName(), e);
+        Thread.currentThread().interrupt();
+      }
     }
     regionThreads.remove(rst);
     return rst.getName();
@@ -369,7 +373,7 @@ public List<JVMClusterUtil.MasterThread> getLiveMasters() {
    * Wait for the specified master to stop. Removes this thread from list of running threads.
    * @return Name of master that just went down.
    */
-  public String waitOnMaster(int serverNumber) throws InterruptedException {
+  public String waitOnMaster(int serverNumber) {
     JVMClusterUtil.MasterThread masterThread = this.masterThreads.get(serverNumber);
     return waitOnMaster(masterThread);
   }
@@ -378,10 +382,16 @@ public String waitOnMaster(int serverNumber) throws InterruptedException {
    * Wait for the specified master to stop. Removes this thread from list of running threads.
    * @return Name of master that just went down.
    */
-  public String waitOnMaster(JVMClusterUtil.MasterThread masterThread) throws InterruptedException {
+  public String waitOnMaster(JVMClusterUtil.MasterThread masterThread) {
     while (masterThread.isAlive()) {
-      LOG.info("Waiting on " + masterThread.getMaster().getServerName().toString());
-      masterThread.join();
+      try {
+        LOG.info("Waiting on " + masterThread.getMaster().getServerName().toString());
+        masterThread.join();
+      } catch (InterruptedException e) {
+        LOG.error("Interrupted while waiting for {} to finish. Retrying join",
+            masterThread.getName(), e);
+        Thread.currentThread().interrupt();
+      }
     }
     masterThreads.remove(masterThread);
     return masterThread.getName();

hbase-server/src/test/java/org/apache/hadoop/hbase/SingleProcessHBaseCluster.java

@@ -19,7 +19,6 @@
 package org.apache.hadoop.hbase;
 
 import java.io.IOException;
-import java.io.InterruptedIOException;
 import java.security.PrivilegedAction;
 import java.util.ArrayList;
 import java.util.HashSet;
@@ -298,11 +297,7 @@ public void resumeRegionServer(ServerName serverName) throws IOException {
   @Override
   public void waitForRegionServerToStop(ServerName serverName, long timeout) throws IOException {
     // ignore timeout for now
-    try {
-      waitOnRegionServer(getRegionServerIndex(serverName));
-    } catch (InterruptedException e) {
-      throw (InterruptedIOException) new InterruptedIOException().initCause(e);
-    }
+    waitOnRegionServer(getRegionServerIndex(serverName));
   }
 
   @Override
@@ -398,11 +393,7 @@ public void stopMaster(ServerName serverName) throws IOException {
   @Override
   public void waitForMasterToStop(ServerName serverName, long timeout) throws IOException {
     // ignore timeout for now
-    try {
-      waitOnMaster(getMasterIndex(serverName));
-    } catch (InterruptedException e) {
-      throw (InterruptedIOException) new InterruptedIOException().initCause(e);
-    }
+    waitOnMaster(getMasterIndex(serverName));
   }
 
   /**
@@ -518,7 +509,7 @@ public JVMClusterUtil.RegionServerThread resumeRegionServer(int serverNumber) {
    * Wait for the specified region server to stop. Removes this thread from list of running threads.
    * @return Name of region server that just went down.
    */
-  public String waitOnRegionServer(final int serverNumber) throws InterruptedException {
+  public String waitOnRegionServer(final int serverNumber) {
     return this.hbaseCluster.waitOnRegionServer(serverNumber);
   }
 
@@ -610,7 +601,7 @@ public JVMClusterUtil.MasterThread stopMaster(int serverNumber, final boolean sh
    * Wait for the specified master to stop. Removes this thread from list of running threads.
    * @return Name of master that just went down.
    */
-  public String waitOnMaster(final int serverNumber) throws InterruptedException {
+  public String waitOnMaster(final int serverNumber) {
     return this.hbaseCluster.waitOnMaster(serverNumber);
   }
 

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

@@ -70,7 +70,7 @@ public static void teardown() throws Exception {
   }
 
   @Test
-  public void testInfo() throws InterruptedException {
+  public void testInfo() {
     HMaster master = TEST_UTIL.getHBaseCluster().getMaster();
     MetricsMasterWrapperImpl info = new MetricsMasterWrapperImpl(master);
     assertEquals(

hbase-testing-util/src/main/java/org/apache/hadoop/hbase/MiniHBaseCluster.java

@@ -19,7 +19,6 @@
 package org.apache.hadoop.hbase;
 
 import java.io.IOException;
-import java.io.InterruptedIOException;
 import java.security.PrivilegedAction;
 import java.util.ArrayList;
 import java.util.HashSet;
@@ -311,11 +310,7 @@ public void resumeRegionServer(ServerName serverName) throws IOException {
   @Override
   public void waitForRegionServerToStop(ServerName serverName, long timeout) throws IOException {
     //ignore timeout for now
-    try {
-      waitOnRegionServer(getRegionServerIndex(serverName));
-    } catch (InterruptedException e) {
-      throw (InterruptedIOException) new InterruptedIOException().initCause(e);
-    }
+    waitOnRegionServer(getRegionServerIndex(serverName));
   }
 
   @Override
@@ -411,11 +406,7 @@ public void stopMaster(ServerName serverName) throws IOException {
   @Override
   public void waitForMasterToStop(ServerName serverName, long timeout) throws IOException {
     //ignore timeout for now
-    try {
-      waitOnMaster(getMasterIndex(serverName));
-    } catch (InterruptedException e) {
-      throw (InterruptedIOException) new InterruptedIOException().initCause(e);
-    }
+    waitOnMaster(getMasterIndex(serverName));
   }
 
   /**
@@ -544,7 +535,7 @@ public JVMClusterUtil.RegionServerThread resumeRegionServer(int serverNumber) {
    * @param serverNumber
    * @return Name of region server that just went down.
    */
-  public String waitOnRegionServer(final int serverNumber) throws InterruptedException {
+  public String waitOnRegionServer(final int serverNumber) {
     return this.hbaseCluster.waitOnRegionServer(serverNumber);
   }
 
@@ -649,7 +640,7 @@ public JVMClusterUtil.MasterThread stopMaster(int serverNumber,
    * @param serverNumber
    * @return Name of master that just went down.
    */
-  public String waitOnMaster(final int serverNumber) throws InterruptedException {
+  public String waitOnMaster(final int serverNumber) {
     return this.hbaseCluster.waitOnMaster(serverNumber);
   }