felipecrv commented on code in PR #35345:
URL: https://github.com/apache/arrow/pull/35345#discussion_r1383653009
##########
cpp/src/arrow/array/array_nested.h:
##########
@@ -120,10 +135,14 @@ class ARROW_EXPORT ListArray : public
BaseListArray<ListType> {
/// This function does the bare minimum of validation of the offsets and
/// input types, and will allocate a new offsets array if necessary (i.e. if
/// the offsets contain any nulls). If the offsets do not have nulls, they
- /// are assumed to be well-formed
+ /// are assumed to be well-formed.
///
- /// Offsets of an Array's null bitmap can be present or an explicit
- /// null_bitmap, but not both.
+ /// If a null_bitmap is not provided, the nulls will be inferred from the
offsets' or
+ /// sizes' null bitmap. Only one of these two is allowed to have a null
bitmap. But if a
+ /// null_bitmap is provided, the offsets array and the sizes array can't
have nulls.
Review Comment:
Copy and pasta mistake. Fixing now.
--
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]
