Hello Aman Sinha, Fang-Yu Rao, Jason Fehr, Joe McDonnell, Michael Smith, Impala
Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23929
to look at the new patch set (#5).
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: The Tpcds test was moved to the
planner directory to be more consistent with the other tests.
Change-Id: I3797acf0f23da4f94098e2f8f5f7aa378c9ae91a
---
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
M
java/calcite-planner/src/test/java/org/apache/impala/planner/TestCalciteStats.java
M
java/calcite-planner/src/test/java/org/apache/impala/planner/TestReduceExprShuttle.java
R
java/calcite-planner/src/test/java/org/apache/impala/planner/TpcdsCpuCostPlannerTest.java
12 files changed, 57 insertions(+), 1,052 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/29/23929/5
--
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: newpatchset
Gerrit-Change-Id: I3797acf0f23da4f94098e2f8f5f7aa378c9ae91a
Gerrit-Change-Number: 23929
Gerrit-PatchSet: 5
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]>
