HBASE-22354 master never sets abortRequested, and thus abort timeout doesn't work for it

Signed-off-by: Peter Somogyi <psomogyi@apache.org>

Author: sershe-apache

hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java

@@ -2954,6 +2954,7 @@ public void abort(String reason, Throwable cause) {
     if (isAborted() || isStopped()) {
       return;
     }
+    setAbortRequested();
     if (cpHost != null) {
       // HBASE-4014: dump a list of loaded coprocessors.
       LOG.error(HBaseMarkers.FATAL, "Master server abort: loaded coprocessors are: " +

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

@@ -2415,7 +2415,7 @@ public void abort(String reason, Throwable cause) {
     } else {
       LOG.error(HBaseMarkers.FATAL, msg);
     }
-    this.abortRequested = true;
+    setAbortRequested();
     // HBASE-4014: show list of coprocessors that were loaded to help debug
     // regionserver crashes.Note that we're implicitly using
     // java.util.HashSet's toString() method to print the coprocessor names.
@@ -2449,6 +2449,10 @@ public void abort(String reason, Throwable cause) {
     stop(reason, true, null);
   }
 
+  protected final void setAbortRequested() {
+    this.abortRequested = true;
+  }
+
   /**
    * @see HRegionServer#abort(String, Throwable)
    */