pixelherodev commented on PR #543:
URL: https://github.com/apache/arrow-go/pull/543#issuecomment-3453082965

   I think I'm going to split this into a few small PRs if that makes sense:
   
   - Regenerate the flatbuf package without any changes from me (it's out of 
sync with the arrow/ repo)
   - As a follow up to that, regenerate it with patches to remove unnecessary 
allocations, and document in the headers that the generated code has been 
modified to remove allocations, and make the changes to e.g. initFB and our 
usage of it to avoid heap escapes
   
   - Separately, the internal changes to the IPC that just clean up buffer 
management (preallocation / reuse)
   
   - Separately, the miscellaneous small changes that don't fit in
   
   That okay?


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