stevenzwu commented on PR #14502: URL: https://github.com/apache/iceberg/pull/14502#issuecomment-3783144488
I agree with the general sentiment that snapshot summary shouldn't include partition summary that can contain sensitive partition column values. In case of encryption, metadata.json is not encrypted. so potentially sensitive information like partition summary shouldn't be added in snapshot summary and metadata.json file. I also didn't see the partition summary field in the spec's implementation notes https://iceberg.apache.org/spec/#optional-snapshot-summary-fields -- 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]
