Steve Carlin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22104 )
Change subject: IMPALA-13524: Calcite planner: support for functions in exprs.test ...................................................................... IMPALA-13524: Calcite planner: support for functions in exprs.test Specifically, this commit supports translate, the not operator (!), the null not equals operator and some "not regexp" functions. A little extra code had to be added for the not regexp functions because Calcite treats the "not" part as an attribute to the operator. Also, added support for "is distinct from" and "is not distinct from" operators which required a convertlet change to handle the conversion from SqlNodes to RexNodes. Change-Id: Ib8c5d5719a409a32ddb6946d1a87c77773f20820 Reviewed-on: http://gerrit.cloudera.org:8080/22104 Reviewed-by: Aman Sinha <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M java/calcite-planner/src/main/codegen/templates/Parser.jj 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/functions/RexCallConverter.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/operators/ImpalaConvertletTable.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/operators/ImpalaOperatorTable.java 5 files changed, 59 insertions(+), 19 deletions(-) Approvals: Aman Sinha: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22104 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib8c5d5719a409a32ddb6946d1a87c77773f20820 Gerrit-Change-Number: 22104 Gerrit-PatchSet: 6 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: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
