dawidwys commented on PR #23090:
URL: https://github.com/apache/flink/pull/23090#issuecomment-1653538853

   But neither `DateTimeUtils` nor `FileSystemTableFactory` use the time zone 
setting from `TableConfig`.
   
   * `FileSystemTableFactory` uses: 
`SINK_PARTITION_COMMIT_WATERMARK_TIME_ZONE`, which has separate validation and 
doesn't go through `ZoneId`.
   * `DateTimeUtils` are used in builtin functions which is applied on incoming 
data.
   
   I am not insisting on making the change now though...


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