Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22029 )
Change subject: IMPALA-13511: Calcite planner: support sub-second datetime parts ...................................................................... IMPALA-13511: Calcite planner: support sub-second datetime parts This commit allows support for milliseconds, microseconds, and nanoseconds for Calcite. The ImpalaSqlIntervalQualifier class extends the Calcite SqlIntervalQualifier class which handles most datetime parts. Some code was copied from the base class since some of the methods were private, but in general, the code handling the parts is exactly the same. Change-Id: I392c3900c70e7754a35ef25fc720ba4a2f2e5dd6 Reviewed-on: http://gerrit.cloudera.org:8080/22029 Reviewed-by: Michael Smith <[email protected]> Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M java/calcite-planner/src/main/codegen/config.fmpp M java/calcite-planner/src/main/codegen/templates/Parser.jj M java/calcite-planner/src/main/java/org/apache/impala/calcite/coercenodes/CoerceOperandShuttle.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/operators/ImpalaOperatorTable.java A java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java 5 files changed, 273 insertions(+), 46 deletions(-) Approvals: Michael Smith: Looks good to me, but someone else must approve Joe McDonnell: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22029 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I392c3900c70e7754a35ef25fc720ba4a2f2e5dd6 Gerrit-Change-Number: 22029 Gerrit-PatchSet: 5 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Anonymous Coward (816) 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]>
