[GitHub] [iceberg] dramaticlly commented on a diff in pull request #8003: Store lastUpdated timestamp at microsecond precision in PartitionsTable

["dramaticlly (via GitHub)" <gi...@apache.org>]

dramaticlly commented on code in PR #8003:
URL: https://github.com/apache/iceberg/pull/8003#discussion_r1254799540


##########
core/src/main/java/org/apache/iceberg/PartitionsTable.java:
##########
@@ -129,7 +130,7 @@ private DataTask task(StaticTableScan scan) {
                   root.posDeleteFileCount,
                   root.eqDeleteRecordCount,
                   root.eqDeleteFileCount,
-                  root.lastUpdatedMs,
+                  Optional.ofNullable(root.lastUpdatedAt).map(t -> t * 1000).orElse(null),

Review Comment:
   yes Szehon, I am not changing how we read from partition metadata table, But the data is stored at microsecond precision instead (if needed to expose via other means)



-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org