rdblue commented on code in PR #18503: URL: https://github.com/apache/iceberg/pull/18503#discussion_r4459497820
########## format/spec.md: ########## @@ -1269,6 +1271,8 @@ The snapshot summary's `operation` field is used by some operations, like snapsh For other optional snapshot summary fields, see [Appendix F](#optional-snapshot-summary-fields). +For encrypted v4 tables, the encryption key for the snapshot root file may be referenced by `key-id` or stored directly in the `encryption-key` field. Review Comment: This pretends that there was an adopted encryption spec/scheme before v4, which is not true. We do not need to note how v4 changes things. -- 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]
