You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zg...@apache.org on 2019/02/20 08:09:56 UTC

[hbase] branch branch-2.1 updated: HBASE-21932 Use Runtime.getRuntime().halt to terminate regionserver when abort timeout

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

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


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new 97445d9  HBASE-21932 Use Runtime.getRuntime().halt to terminate regionserver when abort timeout
97445d9 is described below

commit 97445d99bea5840462c04365253c64e6519d4f7f
Author: Guanghao Zhang <zg...@apache.org>
AuthorDate: Wed Feb 20 16:01:26 2019 +0800

    HBASE-21932 Use Runtime.getRuntime().halt to terminate regionserver when abort timeout
---
 .../java/org/apache/hadoop/hbase/regionserver/HRegionServer.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
index e801d84..cfb3c18 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
@@ -3774,9 +3774,11 @@ public class HRegionServer extends HasThread implements
   private static class SystemExitWhenAbortTimeout extends TimerTask {
     @Override
     public void run() {
-      LOG.warn("Aborting region server timed out, terminating forcibly. Thread dump to stdout.");
+      LOG.warn("Aborting region server timed out, terminating forcibly" +
+          " and does not wait for any running shutdown hooks or finalizers to finish their work." +
+          " Thread dump to stdout.");
       Threads.printThreadInfo(System.out, "Zombie HRegionServer");
-      System.exit(1);
+      Runtime.getRuntime().halt(1);
     }
   }
 }