docete commented on issue #10105: [Flink-14599][table-planner-blink] Support precision of TimestampType in blink planner URL: https://github.com/apache/flink/pull/10105#issuecomment-553294278 @KurtYoung The Date plus Day Interval case and Cast Timestamp to BigInt case, I prefer leave it to the next PR. The semantic of arithmetic and cast operator for date time type need to be double check. Mix the change of only one or two cases in the PR is incomplete. By the way, the TIMESTAMPADD function semantic for DATE +/- INTERVAL_DAY_TIME(hours, minutes and seconds) seems OK to return a TIMESTAMP. What do you think?
---------------------------------------------------------------- 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
