Andrew Wong 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: > Patch Set 5: > > > Patch Set 5: > > > > Sorry for dropping the ball on moving along this review. Would you like to > > rebase this and we can try to get it merged? > > No worries, I tabled the work a bit myself, but I'd be happy to rebase this > later this week. I'm taking the next few days off so I probably won't get to this until next week. 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? -- 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: Sun, 27 Jun 2021 06:33:39 +0000 Gerrit-HasComments: No
