From 5e7afa768e6842f807b95685610ceee7d3af5149 Mon Sep 17 00:00:00 2001 From: Jibing-Li <64681310+Jibing-Li@users.noreply.github.com> Date: Fri, 8 Dec 2023 20:42:17 +0800 Subject: [PATCH] [fix](statistics)Avoid potential NPE #28147 --- .../src/main/java/org/apache/doris/qe/StmtExecutor.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java index 966635a4cb..50af27f47f 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java +++ b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java @@ -2021,7 +2021,9 @@ public class StmtExecutor { */ throwable = t; } finally { - coord.close(); + if (coord != null) { + coord.close(); + } finalizeQuery(); } @@ -2725,7 +2727,9 @@ public class StmtExecutor { throw new RuntimeException("Failed to fetch internal SQL result. " + Util.getRootCauseMessage(e), e); } } finally { - coord.close(); + if (coord != null) { + coord.close(); + } AuditLogHelper.logAuditLog(context, originStmt.toString(), parsedStmt, getQueryStatisticsForAuditLog(), true); QeProcessorImpl.INSTANCE.unregisterQuery(context.queryId());