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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
