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: [email protected] With regards, Apache Git Services
