jorisvandenbossche commented on PR #367: URL: https://github.com/apache/arrow-nanoarrow/pull/367#issuecomment-1900533892
@paleolimbot the reason that the Python binding for the ArrayView currently checks the buffer types dynamically to determine the number of buffers, is that so this works on an uninitialized ArrayView (just based on the layout, without an Array already being created)? See here: https://github.com/apache/arrow-nanoarrow/blob/b3c952a3e21c2b47df85dbede3444f852614a3e2/python/src/nanoarrow/_lib.pyx#L685-L690 i.e. why not just return `self._ptr.array.n_buffers` here? Because if we don't consider the variadic buffers as part of the ArrayView's buffers, then accessing like the above in Python doesn't work for those -- 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]
