You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ne...@apache.org on 2023/11/28 02:22:53 UTC
(doris) branch master updated: (session) fix NereidsTracer shouldLog always true after set enable_nereids_trace from true to false (#27420)
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 <zx...@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 <zx...@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-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org