fallintoplace opened a new pull request, #1512: URL: https://github.com/apache/iceberg-go/pull/1512
## Summary - require `snapshot-id` and `type` when decoding snapshot references - reject non-positive retention values - reject branch-only retention settings on tags during decoding and metadata updates ## Why The Java `SnapshotRef` builder and parser enforce these invariants, but Go only checked the reference type while decoding. Programmatic metadata updates also allowed tag references with `min-snapshots-to-keep` or `max-snapshot-age-ms`, producing metadata Java rejects. ## Testing - `go test ./...` - `go test -race ./table` - `go vet ./...` -- 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]
