Steve Carlin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21923


Change subject: IMPALA-13441: Support explain statements in Impala planner
......................................................................

IMPALA-13441: Support explain statements in Impala planner

This adds support for explain statements for the Calcite planner.
This also fixes up the Parser.jj file so that statements not
processed by Calcite planner will fail, like "describe" and other
non-select statements.  The parser will now only handle "select"
and "explain" as the first keyword.

If the parser fails, we need to do an additional check within Impala.
We run the statement through the Impala parser and check the statement
type.  If the statement type is anything other than SelectStmt,
we run the query within the original Impala planner. If it is a SelectStmt,
we fail the query because we want all select statements to go through
the Calcite parser.

Change-Id: Iea6afaa1f1698a300ad047c8820691cf7e8eb44b
---
M fe/src/main/java/org/apache/impala/planner/Planner.java
M fe/src/main/java/org/apache/impala/service/Frontend.java
M java/calcite-planner/src/main/codegen/templates/Parser.jj
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteJniFrontend.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/ExecRequestCreator.java
M testdata/workloads/functional-query/queries/QueryTest/calcite.test
6 files changed, 144 insertions(+), 104 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/23/21923/1
--
To view, visit http://gerrit.cloudera.org:8080/21923
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iea6afaa1f1698a300ad047c8820691cf7e8eb44b
Gerrit-Change-Number: 21923
Gerrit-PatchSet: 1
Gerrit-Owner: Steve Carlin <[email protected]>

Reply via email to