anesterenok commented on issue #12266: URL: https://github.com/apache/iceberg/issues/12266#issuecomment-2916672959
While I quite in favor of deprecating int96 storage, I must say that its only a projection of Iceberg types. If one uses Iceberg `timestamp` type, the value must be considered a local timestamp and processed as `LocalDateTime`. If one uses Iceberg `timestamptz` type, the value must be considered an instant and processed as `OffsetDateTime` (well, you'd better used `Instant` actually, as no zone offset could really be stored for `timestamptz`). Hence, instead of TimestampInt96Reader: `OffsetDateTime` there should actually have been 2 readers: -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
