Wes McKinney has posted comments on this change. ( http://gerrit.cloudera.org:8080/15661 )
Change subject: wip python: support for passing arrow data to python ...................................................................... Patch Set 5: > That said, I started looking into handling empty columnar scan batches and I > don't see anything specific to empty ArrowArrays in the C data interface > spec. Just wanted to check to make sure I'm not missing it: do we need to set > pointers for the children of an ArrowArray even if the length is 0? If there would be children when the array has non-zero length, yes, I am pretty sure they need to be set, too. This might be something that could be raised in Arrow as a possible relaxation / simplification -- To view, visit http://gerrit.cloudera.org:8080/15661 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5df4714bf5ac339d675f2a9169f8aeff06d30eed Gerrit-Change-Number: 15661 Gerrit-PatchSet: 5 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Wes McKinney <[email protected]> Gerrit-Comment-Date: Tue, 29 Jun 2021 09:35:24 +0000 Gerrit-HasComments: No
