kevinjqliu opened a new pull request, #11354: URL: https://github.com/apache/iceberg/pull/11354
This PR adds additional constraint in the `SnapshotParser` and also includes tests to verify `Snapshot`'s `summary` field is optional in V1 but required in V2. See https://iceberg.apache.org/spec/#snapshots When serializing `Snapshot` to `JSON`, if the `summary` field exists, the `operation` key must also exist ([code](https://github.com/apache/iceberg/blob/main/core/src/main/java/org/apache/iceberg/SnapshotParser.java#L63-L66)). In other words, the `summary` field without the `operation` key is considered invalid. Devlist discussion https://lists.apache.org/thread/h9qmrmlgxh91ol0y2v8olt90b9q6p9xr Relates to [apache/iceberg-python/#1106](https://github.com/apache/iceberg-python/issues/1106) -- 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
