diff --git a/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerReader.java b/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerReader.java index b59e298c0e..6918311410 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerReader.java +++ b/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerReader.java @@ -126,7 +126,9 @@ public class BrokerReader { } catch (TException e) { LOG.warn("Broker close reader failed. fd={}, address={}", fd.toString(), address, e); } - if (tOperationStatus == null || tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { + if (tOperationStatus == null) { + LOG.warn("Broker close reader failed. fd={}, address={}", fd.toString(), address); + } else if (tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { LOG.warn("Broker close reader failed. fd={}, address={}, error={}", fd.toString(), address, tOperationStatus.getMessage()); } diff --git a/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java b/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java index f335701d4a..3bbb8f030d 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java +++ b/fe/fe-core/src/main/java/org/apache/doris/common/util/BrokerUtil.java @@ -263,7 +263,7 @@ public class BrokerUtil { LOG.warn("Broker close reader failed. path={}, address={}", path, address, ex); } } - if (tOperationStatus == null || tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { + if (tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { LOG.warn("Broker close reader failed. path={}, address={}, error={}", path, address, tOperationStatus.getMessage()); } else { @@ -564,7 +564,9 @@ public class BrokerUtil { LOG.warn("Broker close writer failed. filePath={}, address={}", brokerFilePath, address, ex); } } - if (tOperationStatus == null || tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { + if (tOperationStatus == null) { + LOG.warn("Broker close reader failed. fd={}, address={}", fd.toString(), address); + } else if (tOperationStatus.getStatusCode() != TBrokerOperationStatusCode.OK) { LOG.warn("Broker close writer failed. filePath={}, address={}, error={}", brokerFilePath, address, tOperationStatus.getMessage()); } else {