fenfeng9 commented on PR #50166: URL: https://github.com/apache/arrow/pull/50166#issuecomment-4799047625
> Ok, in #50172 the consensus is that null variadic buffers should be allowed, at least when importing from the C Data Interface. > > So we can either: > > * detect null variadic buffers when importing from the C Data Interface, and normalize them as empty non-null buffers > * or allow null variadic buffers everywhere in Arrow C++ (which may be more tedious and more fragile, as we would have to test for this condition explicitly) Agree. Then we just need to check for `nullptr variadic buffers` during Export and set their sizes to `0` instead of calling `buf->size()`. https://github.com/apache/arrow/blob/445851a3b3b22891977ac1a8b5ceabf42e6bbf52/cpp/src/arrow/c/bridge.cc#L606-L614 -- 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]
