dramaticlly opened a new pull request, #8003: URL: https://github.com/apache/iceberg/pull/8003
Good callout from @ajantha-bhat on iceberg spec where `timestamptz` supposed to store at microseconds precision but what we did here is essentially store `lastUpdatedMs` in PartitionsTable with microsecond * 1000, or second level precision. <img width="810" alt="image" src="https://github.com/apache/iceberg/assets/5961173/787a4c61-b4ec-473d-98e2-7ee3626c73d6";> However, this is does not mean we are conforming to the spec as we get such last updated timestamp from iceberg snapshot and it will provide millisecond precious instead of microsecon, so I believe this is best I can think of at the moment. This also tried to rename the column name from `last_updated_ms` to `last_updated_at` to conform with what we had in SnapshotsTable -- 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]
