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

   ### Rationale for this change
   
   Schema.FieldIndices and StructType.FieldIndices return slices backed by 
internal index maps. Mutating a returned slice can corrupt later field lookups.
   
   ### What changes are included in this PR?
   
   Return cloned index slices from both methods. Schema.HasField reads the 
internal map directly so the ownership fix does not add an allocation there.
   
   ### Are these changes tested?
   
   Yes. Regression tests mutate returned slices and verify that Schema and 
StructType keep their original indices. The full arrow package suite passes.
   
   ### Are there any user-facing changes?
   
   Callers now receive slices they can safely modify.


-- 
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]

Reply via email to