Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22475
Change subject: [PROTOTYPE] IMPALA-13653: Create hooks for Calcite planner in Frontend ...................................................................... [PROTOTYPE] IMPALA-13653: Create hooks for Calcite planner in Frontend This is a patch to try organizing the hooks in a FrontendImplFactory. The FrontendImplFactory contains integration points for parsing a statement, getting an AnalysisContext to analyze the statement, and getting a single node planner. This also moves fallback to Frontend::createExecRequest() so that we are invoking the entire compilation process with a single FrontendImplFactory. The fallback behaves exactly the same as it would if Calcite were not present. It currently does "real" fallback for any error, but it will need support for the current testing behavior of falling back for any non-query statement. This ignores the frontend tests at the moment, only changing things to get them to compile. Change-Id: I62a1e669b7a7b2e7cfdf3edafa59942fe06fd825 --- M fe/src/main/java/org/apache/impala/analysis/AnalysisContext.java A fe/src/main/java/org/apache/impala/analysis/ParsedStatement.java A fe/src/main/java/org/apache/impala/analysis/ParsedStatementImpl.java M fe/src/main/java/org/apache/impala/analysis/StmtMetadataLoader.java M fe/src/main/java/org/apache/impala/planner/Planner.java M fe/src/main/java/org/apache/impala/planner/SingleNodePlanner.java A fe/src/main/java/org/apache/impala/planner/SingleNodePlannerIntf.java M fe/src/main/java/org/apache/impala/service/Frontend.java A fe/src/main/java/org/apache/impala/service/FrontendImplFactory.java A fe/src/main/java/org/apache/impala/service/ImpalaFrontendImplFactory.java M fe/src/test/java/org/apache/impala/analysis/ExprRewriteRulesTest.java M fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/common/FrontendFixture.java M fe/src/test/java/org/apache/impala/common/FrontendTestBase.java M fe/src/test/java/org/apache/impala/common/QueryFixture.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteAnalysisContext.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteFrontendImplFactory.java 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/CalciteMetadataHandler.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteParsedStatement.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalcitePhysPlanCreator.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteQueryParser.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteRelNodeConverter.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteSingleNodePlanner.java 25 files changed, 946 insertions(+), 113 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/75/22475/1 -- To view, visit http://gerrit.cloudera.org:8080/22475 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I62a1e669b7a7b2e7cfdf3edafa59942fe06fd825 Gerrit-Change-Number: 22475 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
