This is an automated email from the ASF dual-hosted git repository.

nextdreamblue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new c83e3318a87 (session) fix NereidsTracer shouldLog always true after 
set enable_nereids_trace from true to false (#27420)
c83e3318a87 is described below

commit c83e3318a874aeefe9b9bb09b0de63d642c0b757
Author: xueweizhang <zxw520bl...@163.com>
AuthorDate: Tue Nov 28 10:22:46 2023 +0800

    (session) fix NereidsTracer shouldLog always true after set 
enable_nereids_trace from true to false (#27420)
    
    Signed-off-by: nextdreamblue <zxw520bl...@163.com>
---
 fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java | 2 ++
 .../main/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 63aadc7af19..d50d2d445c9 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: <? 
extends LogicalPlanAdapter>");
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 b28ff8d764d..68392154a62 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;
+    }
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to