fallintoplace opened a new pull request, #1544: URL: https://github.com/apache/iceberg-go/pull/1544
## What changed - Clone key metadata returned by ManifestFile. - Deep-copy partition summaries, including bounds and nullable ContainsNaN values. - Copy the nullable first row ID before returning it. - Reuse the partition summary copy helper for builder inputs and getter outputs. ## Why ManifestFile getters exposed slices and pointers owned by decoded manifest-list entries. A caller could mutate key metadata, partition bounds, summary flags, or first row IDs and silently change later planning or manifest rewrite behavior. The regression test mutates every affected getter and verifies that subsequent reads remain unchanged. ## 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]
