From 04d08254b536efa5a3763880d73015452dc2e8d7 Mon Sep 17 00:00:00 2001
From: Duo Zhang <zhangduo@apache.org>
Date: Sun, 14 May 2023 23:45:36 +0800
Subject: [PATCH] HBASE-27865 TestThriftServerCmdLine fails with
 org.apache.hadoop.hbase.SystemExitRule

---
 .../org/apache/hadoop/hbase/thrift/ThriftServer.java   | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServer.java b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServer.java
index 33bd2ae869f2..81887034aea9 100644
--- a/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServer.java
+++ b/hbase-thrift/src/main/java/org/apache/hadoop/hbase/thrift/ThriftServer.java
@@ -833,9 +833,9 @@ public int run(String[] strings) throws Exception {
     } else {
       setupServer();
     }
-    serviceUGI.doAs(new PrivilegedAction<Object>() {
+    return serviceUGI.doAs(new PrivilegedAction<Integer>() {
       @Override
-      public Object run() {
+      public Integer run() {
         try {
           startInfoServer();
           if (httpEnabled) {
@@ -844,15 +844,13 @@ public Object run() {
           } else {
             tserver.serve();
           }
+          return 0;
         } catch (Exception e) {
           LOG.error(HBaseMarkers.FATAL, "Cannot run ThriftServer", e);
-
-          System.exit(-1);
+          return -1;
         }
-        return null;
       }
     });
-    return 0;
   }
 
   public static void main(String[] args) throws Exception {