alamb commented on code in PR #10689:
URL: https://github.com/apache/arrow-rs/pull/10689#discussion_r3789228027
##########
arrow-ord/src/cmp.rs:
##########
@@ -274,6 +274,28 @@ fn compare_op(op: Op, lhs: &dyn Datum, rhs: &dyn Datum) ->
Result<BooleanArray,
ree: r_ree_info.as_ref().map(|(_, info)| info),
};
+ // Equality against an inlined constant is a scan of fixed-width integers
Review Comment:
maybe you could also mention this is a special case for comparing short
strings with the inlined prefix of a view array
--
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]
