docete edited a comment 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-Time 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 semantic of TIMESTAMPADD function 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

Reply via email to