westonpace commented on code in PR #33641:
URL: https://github.com/apache/arrow/pull/33641#discussion_r1085625034
##########
cpp/src/arrow/array/data.cc:
##########
@@ -195,6 +195,7 @@ int GetNumBuffers(const DataType& type) {
case Type::NA:
case Type::STRUCT:
case Type::FIXED_SIZE_LIST:
+ case Type::RUN_END_ENCODED:
return 1;
Review Comment:
I think most of the code in Acero is used to the idea of a validity buffer
not existing if the null count is 0. So I think the easier path would be to
have 0 buffers. But I'm pretty sure I can work around it if we decide it's
easier elsewhere to have one.
--
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]
