xuzifu666 commented on PR #23754: URL: https://github.com/apache/flink/pull/23754#issuecomment-1818454583
> I don't think this is correct. TO_TIMESTAMP is meant to convert char to a timestamp. If you want to convert to a TIMESTAMP_LTZ, you should just use CONVERT_TZ for that. > > > as presto,trino,starrocks: > > SELECT timestamp '2012-10-31 01:00 UTC' AT TIME ZONE 'America/Los_Angeles'; > > This isn't comparable, since this is not using the function emm，if user want to use many timezone in one sql with TO_TIMESTAMP，they can not choose timezone for it. In a global view，they would use udf for it. this is not friendly to platform developers. Add the timezone arguments can aviod it and not bring any risk. @MartijnVisser -- 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
