Rich-T-kid commented on code in PR #12008:
URL: https://github.com/apache/arrow-rs/pull/12008#discussion_r4503271769
##########
arrow-select/src/concat.rs:
##########
@@ -1559,6 +1569,35 @@ mod tests {
assert!((30..40).contains(&values_len), "{values_len}")
}
+ #[test]
+ fn test_concat_oversized_dictionary_slices() {
Review Comment:
Updated both tests to derive their sizes from the cutoff and assert that the
sparse mapping ran. They now cover primitive values, null dictionary values,
and null keys through both concat and interleave. bdd44b6963
--
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]
