Benjamin0313 commented on PR #16665: URL: https://github.com/apache/iceberg/pull/16665#issuecomment-4909543766
Good question — it's actually stronger than UTC: there is no timezone in play at all. Iceberg's `time` type has always been defined in the spec as time-of-day without date or timezone, stored in Parquet as `TIME_MICROS` with `isAdjustedToUTC=false`. On the Spark side, 4.1's `TimeType` is nanoseconds-of-day internally with `java.time.LocalTime` as the external type — also zone-less. Every conversion this PR adds is pure precision scaling (micros ⇔ nanos, ×1000/÷1000); no zone offset is ever applied on any read or write path (Parquet, ORC, Avro). So there's no UTC normalization to get wrong — the type simply never touches a timezone. -- 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]
