asubiotto opened a new issue, #38918:
URL: https://github.com/apache/arrow/issues/38918

   ### Describe the enhancement requested
   
   https://github.com/apache/arrow/pull/35307 changed schema.Fields to return a 
copy of the fields for safety:
   
https://github.com/apache/arrow/blob/c614014db805caf3bbc4d5270bd990c095bf8725/go/arrow/schema.go#L198
   
   However, a big part of the codebase continues to use schema.Fields assuming 
it is zero-allocation. We should review all callsites and change to using 
`NumFields` in cases where we only need the number of fields and looping over 
the field indices rather than the result of `schema.Fields`.
   
   ### Component(s)
   
   Go


-- 
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: issues-unsubscr...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to