wesm commented on a change in pull request #7598:
URL: https://github.com/apache/arrow/pull/7598#discussion_r449192621
##########
File path: cpp/src/arrow/array/array_base.h
##########
@@ -86,16 +86,17 @@ class ARROW_EXPORT Array {
std::shared_ptr<DataType> type() const { return data_->type; }
Type::type type_id() const { return data_->type->id(); }
- /// Buffer for the null bitmap.
+ /// Buffer for the validity (null) bitmap, if any. Note that Union types
+ /// never have a null bitmap.
///
- /// Note that for `null_count == 0`, this can be null.
- /// This buffer does not account for any slice offset
+ /// Note that for `null_count == 0` or for null or union types, this will be
Review comment:
Right, will clarify
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
