Impala Public Jenkins has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/21923 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: Iea6afaa1f1698a300ad047c8820691cf7e8eb44b Gerrit-Change-Number: 21923 Gerrit-PatchSet: 7 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
