You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2023/07/26 17:06:09 UTC
[pinot] branch master updated: removing the format support in explain physical plan sql queries. (#11179)
This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 09bc525598 removing the format support in explain physical plan sql queries. (#11179)
09bc525598 is described below
commit 09bc525598c07bcd27531288dacc265617920c16
Author: Abhishek Sharma <ab...@spothero.com>
AuthorDate: Wed Jul 26 13:06:03 2023 -0400
removing the format support in explain physical plan sql queries. (#11179)
---
pinot-common/src/main/codegen/includes/parserImpls.ftl | 13 +------------
.../java/org/apache/pinot/query/QueryCompilationTest.java | 4 ++--
2 files changed, 3 insertions(+), 14 deletions(-)
diff --git a/pinot-common/src/main/codegen/includes/parserImpls.ftl b/pinot-common/src/main/codegen/includes/parserImpls.ftl
index 79c1a30f44..989894dd5d 100644
--- a/pinot-common/src/main/codegen/includes/parserImpls.ftl
+++ b/pinot-common/src/main/codegen/includes/parserImpls.ftl
@@ -125,22 +125,11 @@ SqlNode SqlPhysicalExplain() :
SqlNode stmt;
SqlExplainLevel detailLevel = SqlExplainLevel.EXPPLAN_ATTRIBUTES;
SqlExplain.Depth depth = SqlExplain.Depth.PHYSICAL;
- final SqlExplainFormat format;
+ final SqlExplainFormat format = SqlExplainFormat.TEXT;
}
{
<EXPLAIN> <IMPLEMENTATION> <PLAN>
[ detailLevel = ExplainDetailLevel() ]
- (
- LOOKAHEAD(2)
- <AS> <XML> { format = SqlExplainFormat.XML; }
- |
- LOOKAHEAD(2)
- <AS> <JSON> { format = SqlExplainFormat.JSON; }
- |
- <AS> <DOT_FORMAT> { format = SqlExplainFormat.DOT; }
- |
- { format = SqlExplainFormat.TEXT; }
- )
<FOR> stmt = SqlQueryOrDml() {
return new SqlPhysicalExplain(getPos(),
stmt,
diff --git a/pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.java b/pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.java
index c10a32eafe..3015deea4b 100644
--- a/pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.java
+++ b/pinot-query-planner/src/test/java/org/apache/pinot/query/QueryCompilationTest.java
@@ -476,7 +476,7 @@ public class QueryCompilationTest extends QueryEnvironmentTestBase {
private Object[][] provideQueriesWithExplainedPhysicalPlan() {
//@formatter:off
return new Object[][] {
-new Object[]{"EXPLAIN IMPLEMENTATION PLAN INCLUDING ALL ATTRIBUTES AS JSON FOR SELECT col1, col3 FROM a",
+new Object[]{"EXPLAIN IMPLEMENTATION PLAN INCLUDING ALL ATTRIBUTES FOR SELECT col1, col3 FROM a",
"[0]@localhost:3 MAIL_RECEIVE(RANDOM_DISTRIBUTED)\n"
+ "├── [1]@localhost:2 MAIL_SEND(RANDOM_DISTRIBUTED)->{[0]@localhost@{3,3}|[0]}\n"
+ "│ └── [1]@localhost:2 PROJECT\n"
@@ -484,7 +484,7 @@ new Object[]{"EXPLAIN IMPLEMENTATION PLAN INCLUDING ALL ATTRIBUTES AS JSON FOR S
+ "└── [1]@localhost:1 MAIL_SEND(RANDOM_DISTRIBUTED)->{[0]@localhost@{3,3}|[0]}\n"
+ " └── [1]@localhost:1 PROJECT\n"
+ " └── [1]@localhost:1 TABLE SCAN (a) null\n"},
-new Object[]{"EXPLAIN IMPLEMENTATION PLAN EXCLUDING ATTRIBUTES AS DOT FOR "
+new Object[]{"EXPLAIN IMPLEMENTATION PLAN EXCLUDING ATTRIBUTES FOR "
+ "SELECT col1, COUNT(*) FROM a GROUP BY col1",
"[0]@localhost:3 MAIL_RECEIVE(RANDOM_DISTRIBUTED)\n"
+ "├── [1]@localhost:2 MAIL_SEND(RANDOM_DISTRIBUTED)->{[0]@localhost@{3,3}|[0]} (Subtree Omitted)\n"
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org