fallintoplace opened a new pull request, #1547:
URL: https://github.com/apache/iceberg-go/pull/1547

   ## What changed
   
   Clone mapped field IDs and name slices while building the result of 
UpdateNameMapping. Fields without IDs now retain the independently visited 
child results as well.
   
   ## Why
   
   The update visitor reused FieldID pointers from the input mapping. Mutating 
the returned mapping could therefore change the original. Appending a renamed 
field to a name slice with spare capacity could also write into the input 
backing array during the update itself.
   
   The regression test covers both alias paths across parent and nested fields.
   
   ## 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]

Reply via email to