Shekharrajak opened a new pull request, #16577:
URL: https://github.com/apache/iceberg/pull/16577
Closes #12751
Switches Iceberg's Avro encoding for non-zone timestamp types
(`timestamp`, `timestamp_ns`) to the Avro-spec-correct
`local-timestamp-{micros,nanos}` logical types, instead of overloading
`timestamp-{micros,nanos}` with the Iceberg-private
`adjust-to-utc=false` convention.
The zone-adjusted variants (`timestamptz`, `timestamptz_ns`) continue
to use `timestamp-{micros,nanos}` with `adjust-to-utc=true`.
## Why
Per the Avro spec, `timestamp-{millis,micros,nanos}` is an instant on
the global timeline (UTC), while `local-timestamp-{millis,micros,nanos}`
is the timezone-free local timestamp.
Iceberg's writer mapped both the
zone and non-zone Iceberg types to `timestamp-*` and disambiguated with
the `adjust-to-utc` property, which the spec footnote already calls
out as an Iceberg-only convention.
--
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]
