kou commented on a change in pull request #11674:
URL: https://github.com/apache/arrow/pull/11674#discussion_r748978283
##########
File path: cpp/src/arrow/array/array_nested.cc
##########
@@ -680,7 +680,7 @@ Result<std::shared_ptr<Array>>
SparseUnionArray::GetFlattenedField(
if (child_null_bitmap) {
BitmapAnd(flattened_null_bitmap->data(), child_offset,
child_null_bitmap->data(),
child_offset, child_data->length, child_offset,
- flattened_null_bitmap->mutable_data());
+ flattened_null_bitmap->mutable_data(), /*out pop count*/
nullptr);
Review comment:
I'm not sure that the problem case correctly but I think that there are
no problem with default parameter.
(Is this ABI compatibility problem?)
Because we increase major version for each release.
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
