danepitkin commented on code in PR #39813:
URL: https://github.com/apache/arrow/pull/39813#discussion_r1481725217
##########
python/pyarrow/array.pxi:
##########
@@ -2557,9 +2572,12 @@ cdef class ListViewArray(Array):
Return the underlying array of values which backs the ListViewArray
ignoring the array's offset and sizes.
- If any of the list elements are null, but are backed by a
- non-empty sub-list, those elements will be included in the
- output.
+ The values array may be out of order and/or contain additional values
+ that are not found in the logical representation of the array. The only
+ guarantee is that each non-null value in the ListView Array is
contiguous.
+
+ Compare with :meth:`flatten`, which returns only the non-null
+ values taking into consideration the array's offset.
Review Comment:
+1 I also updated LargeListViewArray.values() docstrings
--
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]
