Steve Carlin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23929 )
Change subject: IMPALA-14541: Remove deprecated CalciteJniFrontend class ...................................................................... IMPALA-14541: Remove deprecated CalciteJniFrontend class The CalciteJniFrontend class is no longer used. The main interface for the Calcite code is now CalciteCompilerFactory which is instantiated by the Frontend class. Various other method and classes only used by the CalciteJniFrontend method have also been deleted. The test framework has been modified to use the CalciteCompilerFactory interface. One minor change also in this commit: Some tests were moved to the calcite/planner directory to be more consistent. Change-Id: I3797acf0f23da4f94098e2f8f5f7aa378c9ae91a Reviewed-on: http://gerrit.cloudera.org:8080/23929 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Steve Carlin <[email protected]> --- M fe/src/test/java/org/apache/impala/common/FrontendTestBase.java D 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 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 D java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteValidator.java D java/calcite-planner/src/main/java/org/apache/impala/calcite/service/ExecRequestCreator.java R java/calcite-planner/src/test/java/org/apache/impala/calcite/planner/CalcitePlannerTest.java R java/calcite-planner/src/test/java/org/apache/impala/calcite/planner/TestCalciteStats.java R java/calcite-planner/src/test/java/org/apache/impala/calcite/planner/TestReduceExprShuttle.java 12 files changed, 71 insertions(+), 1,065 deletions(-) Approvals: Impala Public Jenkins: Verified Steve Carlin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23929 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3797acf0f23da4f94098e2f8f5f7aa378c9ae91a Gerrit-Change-Number: 23929 Gerrit-PatchSet: 8 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
