jorisvandenbossche commented on PR #318:
URL: https://github.com/apache/arrow-nanoarrow/pull/318#issuecomment-1814565944

   > I think you probably want to use the `Schema`, `Array`, and `ArrayStream` 
constructors with the capsule as the `base` instead of "moving" into the 
"holder" helpers.
   
   That's fine for me as well, no strong opinion here. I actually started that 
way, but thought that moving the struct as consumer so we own the memory would 
be considered good practice (nanoarrow is of course a bit of a peculiar 
consumer). But actually for nested ones that still only moves the top-level 
one, so it's not that this guarantees anything ..


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