pitrou commented on PR #50327: URL: https://github.com/apache/arrow/pull/50327#issuecomment-4916577681
> One question before I rework: do you prefer this mechanism in Cython as sketched, or as a C++ `ToPyList` visitor under `python/pyarrow/src/arrow/python/` (in the spirit of `MonthDayNanoIntervalArrayToPyList`)? The Cython version keeps the unspecialized fallback trivially exact and the diff reviewable; the C++ visitor is likely faster still (and would also serve `ChunkedArray`/`Table` uniformly) but is a much bigger surface. Happy to go either way. Doing this in Cython would probably be more maintainable, right? Wrangling CPython API calls from C++ is generally tedious and error-prone. -- 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]
