ajantha-bhat commented on PR #6090: URL: https://github.com/apache/iceberg/pull/6090#issuecomment-1306046322
> I think we should change the label of the snapshot-id entry in https://iceberg.apache.org/spec/#table-statistics (to level, not blob level) Sorry, I still didn't get how the query engine will figure out the statistics file for the current snapshot (when the snapshot is reused). Instead of the suggested change, can we change `statisticsFile.snapshotId()` to the snapshot id of the referring snapshot? This way `TableMetadata` will have entries for each snapshot id (even for the resue case). Snapshot file path can be reused. @rdblue: What do you think about this? -- 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
