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 2020/09/21 00:29:21 UTC

[hbase] branch master updated (8efd250 -> 7731856)

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

zghao pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git.


    from 8efd250  HBASE-24976 Printing the swallowed exception
     new 8c11007  Revert "HBASE-24976 Printing the swallowed exception"
     new 7731856  HBASE-24976 REST Server failes to start without any error message

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:


[hbase] 02/02: HBASE-24976 REST Server failes to start without any error message

Posted by zg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 773185670bae57e044b5d11713cd796dfab1e198
Author: lujiefsi <lu...@foxmail.com>
AuthorDate: Mon Sep 21 08:25:58 2020 +0800

    HBASE-24976 REST Server failes to start without any error message
    
    Signed-off-by: Wellington Chevreuil <wc...@apache.org>
    Signed-off-by: Viraj Jasani <vj...@apache.org>
    Signed-off-by: Guanghao Zhang <zg...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
index 2ad57e1..c6f769e 100644
--- a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
+++ b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
@@ -382,13 +382,8 @@ public class RESTServer implements Constants {
       this.infoServer.setAttribute("hbase.conf", conf);
       this.infoServer.start();
     }
-    try {
-      // start server
-      server.start();
-    } catch (Exception e) {
-      LOG.error(HBaseMarkers.FATAL, "Failed to start server", e);
-      throw e;
-    }
+    // start server
+    server.start();
   }
 
   public synchronized void join() throws Exception {
@@ -442,6 +437,7 @@ public class RESTServer implements Constants {
       server.run();
       server.join();
     } catch (Exception e) {
+      LOG.error(HBaseMarkers.FATAL, "Failed to start server", e);
       System.exit(1);
     }
 


[hbase] 01/02: Revert "HBASE-24976 Printing the swallowed exception"

Posted by zg...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 8c11007b7c1b59c2fd3d6e6aa1e12f997dd90455
Author: Guanghao Zhang <zg...@apache.org>
AuthorDate: Mon Sep 21 08:27:38 2020 +0800

    Revert "HBASE-24976 Printing the swallowed exception"
    
    This reverts commit 8efd2509e9f4d44a699f0ad0e50c0b7ca7ad2b2a.
---
 .../src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
index c6f769e..2ad57e1 100644
--- a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
+++ b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/RESTServer.java
@@ -382,8 +382,13 @@ public class RESTServer implements Constants {
       this.infoServer.setAttribute("hbase.conf", conf);
       this.infoServer.start();
     }
-    // start server
-    server.start();
+    try {
+      // start server
+      server.start();
+    } catch (Exception e) {
+      LOG.error(HBaseMarkers.FATAL, "Failed to start server", e);
+      throw e;
+    }
   }
 
   public synchronized void join() throws Exception {
@@ -437,7 +442,6 @@ public class RESTServer implements Constants {
       server.run();
       server.join();
     } catch (Exception e) {
-      LOG.error(HBaseMarkers.FATAL, "Failed to start server", e);
       System.exit(1);
     }