This is an automated email from the ASF dual-hosted git repository.
huajianlan pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new e249e00586a [fix](Nereids) fix explain plan with sql cache (#39431)
(#39463)
e249e00586a is described below
commit e249e00586a0fecffeeb1ea1a1712aba7d2009f9
Author: 924060929 <924060...@qq.com>
AuthorDate: Fri Aug 16 15:51:47 2024 +0800
[fix](Nereids) fix explain plan with sql cache (#39431) (#39463)
introduced by #38950, explain plan with sql cache will throw an exception
```
errCode = 2, detailMessage = Cannot invoke
"org.apache.doris.nereids.trees.plans.Plan.treeString()" because
"this.optimizedPlan" is null
```
---
.../src/main/java/org/apache/doris/nereids/NereidsPlanner.java | 2 +-
regression-test/suites/query_p0/cache/sql_cache.groovy | 7 +++++++
2 files changed, 8 insertions(+), 1 deletion(-)
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 f607b6cf91e..74a54036f9c 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
@@ -174,7 +174,7 @@ public class NereidsPlanner extends Planner {
if (plan instanceof LogicalSqlCache) {
rewrittenPlan = analyzedPlan = plan;
LogicalSqlCache logicalSqlCache = (LogicalSqlCache) plan;
- physicalPlan = new PhysicalSqlCache(
+ optimizedPlan = physicalPlan = new PhysicalSqlCache(
logicalSqlCache.getQueryId(),
logicalSqlCache.getColumnLabels(),
logicalSqlCache.getFieldInfos(),
logicalSqlCache.getResultExprs(),
logicalSqlCache.getResultSetInFe(),
diff --git a/regression-test/suites/query_p0/cache/sql_cache.groovy
b/regression-test/suites/query_p0/cache/sql_cache.groovy
index d4b16cb5438..05ad8f6d81d 100644
--- a/regression-test/suites/query_p0/cache/sql_cache.groovy
+++ b/regression-test/suites/query_p0/cache/sql_cache.groovy
@@ -212,4 +212,11 @@ suite("sql_cache") {
"""
sql "ADMIN SET FRONTEND CONFIG ('cache_last_version_interval_second' =
'10')"
+
+ // explain plan with sql cache
+ connect {
+ sql "set enable_sql_cache=true"
+ sql "select 100"
+ sql "explain plan select 100"
+ }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
