fallintoplace opened a new pull request, #1546: URL: https://github.com/apache/iceberg-go/pull/1546
## What changed Return a fresh literal set from BoundSetPredicate.Literals instead of exposing the set stored by the bound expression. ## Why Set includes the mutating Add method. Returning the internal set allowed callers and expression visitors to inject literals into an already-bound predicate, changing later evaluation and projection results. The regression test mutates a returned set and verifies that subsequent reads still contain only the literals used to construct the predicate. ## Testing - go test . - go vet . - go test ./... -- 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]
