From c83e3318a874aeefe9b9bb09b0de63d642c0b757 Mon Sep 17 00:00:00 2001 From: xueweizhang Date: Tue, 28 Nov 2023 10:22:46 +0800 Subject: [PATCH] (session) fix NereidsTracer shouldLog always true after set enable_nereids_trace from true to false (#27420) Signed-off-by: nextdreamblue --- .../main/java/org/apache/doris/nereids/NereidsPlanner.java | 2 ++ .../java/org/apache/doris/nereids/minidump/NereidsTracer.java | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java index 63aadc7af1..d50d2d445c 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java @@ -98,6 +98,8 @@ public class NereidsPlanner extends Planner { public void plan(StatementBase queryStmt, org.apache.doris.thrift.TQueryOptions queryOptions) { if (statementContext.getConnectContext().getSessionVariable().isEnableNereidsTrace()) { NereidsTracer.init(); + } else { + NereidsTracer.disable(); } if (!(queryStmt instanceof LogicalPlanAdapter)) { throw new RuntimeException("Wrong type of queryStmt, expected: "); diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/minidump/NereidsTracer.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/minidump/NereidsTracer.java index b28ff8d764..68392154a6 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/minidump/NereidsTracer.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/minidump/NereidsTracer.java @@ -176,5 +176,9 @@ public class NereidsTracer { TRACE_PATH = Optional.ofNullable(TRACE_PATH).orElse(Config.nereids_trace_log_dir); new File(TRACE_PATH).mkdirs(); } + + public static void disable() { + NereidsTracer.shouldLog = false; + } }