KurtYoung commented on a change in pull request #10399:
[FLINK-14959][table-planner-blink] Support precision of LocalZonedTimestampType
in blink planner
URL: https://github.com/apache/flink/pull/10399#discussion_r353580510
##########
File path:
flink-table/flink-table-planner-blink/src/main/scala/org/apache/flink/table/planner/codegen/calls/ScalarFunctionCallGen.scala
##########
@@ -136,13 +148,24 @@ object ScalarFunctionCallGen {
}
parameterClasses.zipWithIndex.zip(operands).map { case ((paramClass, i),
operandExpr) =>
+ // Convert TIMESTAMP WITH LOCAL TIME ZONE to Long if the UDX need Long
+ // but the user pass TIMESTAMP WITH LOCAL TIME ZONE instead.
+ val newOperatorExpr =
+ if (operandExpr.resultType.getTypeRoot ==
LogicalTypeRoot.TIMESTAMP_WITH_LOCAL_TIME_ZONE
Review comment:
dito
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
