jorisvandenbossche commented on PR #396: URL: https://github.com/apache/arrow-nanoarrow/pull/396#issuecomment-1983716642
> An early version of this PR implemented the `CMaterializedArrayStream` using C pointers (e.g., `ArrowArray* arrays`), but I decided that was to complex and went back to `List[CArray]`. I think this is also better for managing ownership (e.g., more unneeded `CArray` instances can be released by the garbage collector). If it would at some point turn out to be a perf bottleneck to use an untyped list, we can also consider compiling the cython code to C++ instead of C and then can use things like vector -- 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]
