dannycjones opened a new pull request, #16005: URL: https://github.com/apache/iceberg/pull/16005
This is a minor clarification to be clear that the following fields in snapshot summaries refer to the physical layout (i.e. records in data and delete files) rather than logical records (i.e. active records, that should be considered to be valid to return in queries). It was unclear to me when implementing a compaction algorithm what the values referred to. I have reviewed the Java implementation to confirm that these fields - at least in Java - refer to the physical layout: https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/SnapshotSummary.java#L322-L325. I acknowledge this is changing the specification (`format/` dir) - please let me know if there is a good way to proceed. I'm hoping this is a simple clarification. -- 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]
