alamb commented on code in PR #6306:
URL: https://github.com/apache/arrow-rs/pull/6306#discussion_r1732827235
##########
arrow-array/src/array/byte_view_array.rs:
##########
@@ -317,8 +317,11 @@ impl<T: ByteViewType + ?Sized> GenericByteViewArray<T> {
})
}
- /// Returns an iterator over the prefix bytes of this array with respect
to the prefix length.
- /// If the prefix length is larger than the string length, it will return
the empty slice.
+ /// Returns an iterator over the first `prefix_len` bytes of each array
+ /// element, including null values.
+ ///
+ /// If `prefix_len` is larger than the element's length, the iterator will
Review Comment:
The results on https://github.com/apache/arrow-rs/pull/6312 seem to show
that using null masks does indeed reduce performance
So is the conclusion that these APIs are good to go?
--
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]
