isidentical commented on PR #4039:
URL: 
https://github.com/apache/arrow-datafusion/pull/4039#issuecomment-1296752289

   @tustvold @Dandandan it seems like the limit is never applied for 
multi-column sorts when `lexsort_to_indices` is called. Probably due to the 
changes in https://github.com/apache/arrow-rs/pull/2929, where we previously 
created an array of `(&value_indices)[0..len]` but now just create it from 
`value_indices` directly without any limit (the single-column case is handled 
separately; and it still respects the limit).


-- 
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]

Reply via email to