flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [flink] docete commented on issue #10035: [FLINK-14080][table-planner-blink] Introduce Timestamp as internal representation of TIMESTAMP_WITHOUT_TIME_ZONE
Date Thu, 31 Oct 2019 02:32:39 GMT
docete commented on issue #10035: [FLINK-14080][table-planner-blink] Introduce Timestamp as
internal representation of TIMESTAMP_WITHOUT_TIME_ZONE
URL: https://github.com/apache/flink/pull/10035#issuecomment-548191373
 
 
   Hi @wuchong , we introduce this class for performance reasons. The LocalDateTime is composed
of a LocalDate, which hold year(int), month(short), day(short), and a LocalTime, which holds
hour(byte), minute(byte), second(byte) and nanosecond(int). It's not computer friendly. I
plan to use this class to represent TIMESTAMP_WITHOUT_TIME_ZONE and TIMESTAMP_WITH_LOCAL_TIME_ZONE.
And for TIMESTAMP_WITH_TIME_ZONE, we may introduce another class which holds a Zone offset
as internal representation.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message