[GitHub] [iceberg] zhangdove commented on pull request #1355: Fixed non-greenway time zone, data loss with day partition

[GitBox <gi...@apache.org>]

zhangdove commented on pull request #1355:
URL: https://github.com/apache/iceberg/pull/1355#issuecomment-675874398


   @rdblue Thanks for your reply. And I still have some questions about time zones.
   
   >Timestamp modification or adjustment to display a value by time zone is done by processing engines, not by Iceberg itself. Iceberg must only store values and return the same values.
   
   1.I don't think that such an operation changes the value of the time, and Iceberg does not change the real data of the time type, but only changes the value of the time partition directory to meet the user's current time zone.
   
   2.I don't understand why iceberg must use UTC time zone in the time partition directory. The user's time data in other time zones is prone to the problems described in #1354 .


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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org