[
https://issues.apache.org/jira/browse/FLINK-34147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17810347#comment-17810347
]
Rui Li commented on FLINK-34147:
--------------------------------
I think having two separate classes is fine, but it might be an overkill.
Perhaps we can just add methods that take a time zone parameter? Or we can at
least improve the doc to clarify all conversions are performed in UTC.
> TimestampData to/from LocalDateTime is ambiguous
> ------------------------------------------------
>
> Key: FLINK-34147
> URL: https://issues.apache.org/jira/browse/FLINK-34147
> Project: Flink
> Issue Type: Bug
> Components: Table SQL / API
> Reporter: Rui Li
> Priority: Major
>
> It seems TimestampData is essentially an {{Instant}}. Therefore an implicit
> time zone is used in the {{fromLocalDateTime}} and {{toLocalDateTime}}
> methods. However neither the method name nor the API doc indicates which time
> zone is used. So from caller's perspective, the results of these two methods
> are ambiguous.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
