Hi everyone, I'd like to start a vote for the following REST OpenAPI specification clarification:
PR: https://github.com/apache/iceberg/pull/17800 Thanks @singhpk234 and @gaborkaszab 's reviewing. This change adds the optional key-id field to the OpenAPI Snapshot schema and regenerates the Python models. The field is already supported by SnapshotParser, so this aligns the REST OpenAPI schema with the existing Snapshot representation. The field remains optional, preserving compatibility with snapshots that do not use encryption. There was also a review comment about making the field description more future-proof for V4, where snapshots will use a root manifest file instead of a manifest list file. I’d also like to hear everyone's thoughts on whether we should modify the description now or wait to update it uniformly in v4. Please vote on the proposed change: [ ] +1 Approve the change [ ] +0 No opinion [ ] -1 Do not approve the change because... Thanks, Shangqing
