yiyutian1 commented on PR #25763:
URL: https://github.com/apache/flink/pull/25763#issuecomment-2572428331

   > Thanks for addressing feedback it looks we are approaching
   > 
   > There is at least one case found showing regression
   > 
   > ```sql
   > SELECT to_timestamp_ltz(123, CAST(NULL AS INTEGER));
   > ```
   > 
   > before PR returns `null`
   > 
   > after PR it fails on validation
   
   such good catch! Fixed it in the new commit. thanks for the thorough checks. 


-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to