Fix SHOW BACKENDS return ERROR (#320)

In some case, errMsg in Backend maybe null. we change it when check it nut null Issue: #317
apache · Nov 15, 2018 · 8ac9492 · 8ac9492
1 parent f1b6735
commit 8ac9492
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/fe/src/main/java/org/apache/doris/system/Backend.java b/fe/src/main/java/org/apache/doris/system/Backend.java
@@ -212,7 +212,11 @@ public void setBad(EventBus eventBus, String errMsg) {
         }
 
         eventBus.post(new BackendEvent(BackendEventType.BACKEND_DOWN, "missing heartbeat", Long.valueOf(id)));
-        heartbeatErrMsg = errMsg;
+        // In some case, errMsg is null when catched Exception have no message, which can make
+        // `SHOW BACKENDS` return ERROR. We check errMsg here to avoid.
+        if (errMsg != null) {
+            heartbeatErrMsg = errMsg;
+        }
     }
 
     public void setBackendState(BackendState state) {