codephage2020 commented on code in PR #9347:
URL: https://github.com/apache/arrow-rs/pull/9347#discussion_r2761831391
##########
arrow-ord/src/comparison.rs:
##########
@@ -109,14 +109,96 @@ where
Ok(BooleanArray::new(values, None))
}
+/// Checks if each value in the [`PrimitiveArray`] is contained in the
corresponding [`GenericListViewArray`] element
+pub fn in_list_view<T, OffsetSize>(
Review Comment:
@Jefffrey Thank you! I completely agree that we should prioritize the core
sorting functionality. I'll update the PR .
Regarding the API inconsistency you mentioned with in_list() and
in_list_utf8(), I think that's a valid technical debt issue that should be
discussed separately. Perhaps we should consider unifying these APIs or
designing a more generic contains-style kernel?
--
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]
