Hello Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21525
to look at the new patch set (#2).
Change subject: IMPALA-13136: Refactor AnalyzedFunctionCallExpr (for Calcite)
......................................................................
IMPALA-13136: Refactor AnalyzedFunctionCallExpr (for Calcite)
The analyze method is now called after the Expr is constructed.
This code is more in line with the existing way that Impala
constructs the Expr object.
Change-Id: Ideb662d9c7536659cb558bf62baec29c82217aa2
---
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/AnalyzedFunctionCallExpr.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/AnalyzedNullLiteral.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/RexCallConverter.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/RexLiteralConverter.java
M
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/util/CreateExprVisitor.java
5 files changed, 38 insertions(+), 74 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/21525/2
--
To view, visit http://gerrit.cloudera.org:8080/21525
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ideb662d9c7536659cb558bf62baec29c82217aa2
Gerrit-Change-Number: 21525
Gerrit-PatchSet: 2
Gerrit-Owner: Steve Carlin <scar...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
