fallintoplace opened a new pull request, #1736: URL: https://github.com/apache/iceberg-go/pull/1736
## Summary `ExpireSnapshots` was reading unprefixed retention properties and using defaults that do not match the Iceberg table-property contract. This change: - uses the standard `history.expire.*` property names - matches the standard defaults: 1 snapshot, 5 days, and no ref-age limit - accepts the old unprefixed names as compatibility fallbacks - gives the standard names precedence when both forms are set - updates the retention docs and adds regression coverage ## Test plan - `go test ./... -count=1` - `go test -race ./codec/... ./table/...` - `go vet ./...` - `golangci-lint v2.11.4 run --timeout=10m` -- 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]
