paleolimbot commented on PR #318: URL: https://github.com/apache/arrow-nanoarrow/pull/318#issuecomment-1814781437
> For exporting (the dunder methods), do we want to move in that case? That is a very complicated question, although I think the renaming of classes will help here. For the current `Array` (probably to be renamed `CArray` or `UnsafeArray` to properly communicate the low-level/only-use-if-you-know-what-you-are-doingness of it), I think you can implement `__arrow_c_XXX__` and move to a capsule. For the current `ArrayView` (probably future `Array`), you will want to make a shallow copy. Shallow copying isn't implemented here and is complicated (I do it in R)...I would punt and not do that right now. For Schema, you can use `ArrowSchemaDeepCopy()`, which is probably a safer default than a move. Streams I think you can just move...they are already stateful and so I think there is less opportunity for somebody to misuse them in that way. -- 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]
