Re: [I] Improve timestamp support [iceberg-python]

["kevinjqliu (via GitHub)" <gi...@apache.org>]

kevinjqliu commented on issue #2270:
URL: https://github.com/apache/iceberg-python/issues/2270#issuecomment-3481615151

   hey @bepec 
   
   > But then checked Iceberg specification and it only supports timestamps with tz=UTC - so this example isn't something that should work, is that understanding correct?
   
   Iceberg indeed stores the value of timestamp with timezone in UTC. This is documented in https://iceberg.apache.org/spec/#primitive-types
   
   The engine is expected to convert the timestamp to UTC before storing in Iceberg
   And no, that example should not work. We have an unit test against using timezone other than UTC
   https://github.com/apache/iceberg-python/blob/8878b2c260bf62ad16ef6f8e8d3a6ae93103eff7/tests/io/test_pyarrow_visitor.py#L222-L225
   


-- 
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-unsubscribe@iceberg.apache.org

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