fallintoplace opened a new pull request, #1382: URL: https://github.com/apache/iceberg-go/pull/1382
## Summary `Table` stores the caller-provided identifier slice directly, and `Identifier()` returned that same slice. Because `Identifier` is `[]string`, callers could mutate table identity after construction and affect catalog operations. This patch makes `Table` defensive against aliasing: - `New(...)` now clones the input identifier before storing it. - `Identifier()` now returns a cloned identifier. - Added regression coverage to ensure external identifier mutations cannot affect internal table state. ## Testing - `go test ./table -run TestTableIdentifierIsolation -count=1` - `go test ./table -count=1` -- 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]
