Hello Aman Sinha, Anonymous Coward (816), Joe McDonnell, Michael Smith, Impala
Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21565
to look at the new patch set (#9).
Change subject: IMPALA-13197: Implement Analytic Exprs for Calcite
......................................................................
IMPALA-13197: Implement Analytic Exprs for Calcite
An analytic expression is represented with a RexOver type RexNode
within Calcite. They will exist within the Project RelNode. If
there are any RexOvers existing within the Project, then the
ImpalaAnalyticRel RelNode gets created instead of the ImpalaProjectRel.
Only bare bones test cases are included. There are quite a number
of analytic expressions that will not work. The logic is included in
the AnalyticExpr.standardize() method. Another commit will be needed
to support all general analytic expressions and the tests within
Impala will be used for testing purposes.
Change-Id: Iba5060546a7568ba0cd315f546daa78d89b1c3c5
---
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/coercenodes/CoerceOperandShuttle.java
A
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/AnalyzedAnalyticExpr.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/FunctionResolver.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ConvertToImpalaRelRules.java
A
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaAnalyticRel.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/util/CreateExprVisitor.java
M testdata/workloads/functional-query/queries/QueryTest/calcite.test
7 files changed, 664 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/65/21565/9
--
To view, visit http://gerrit.cloudera.org:8080/21565
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iba5060546a7568ba0cd315f546daa78d89b1c3c5
Gerrit-Change-Number: 21565
Gerrit-PatchSet: 9
Gerrit-Owner: Steve Carlin <scar...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward (816)
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
