sundy-li commented on pull request #9602: URL: https://github.com/apache/arrow/pull/9602#issuecomment-797294233
Hi, all. From other user's adivice, partial_sort could be ``` pub fn partial_sort<T, F>(v: &mut [T], limit: usize, mut is_less: F) where F: FnMut(&T, &T) -> Ordering, { let (before, _mid, _after) = v.select_nth_unstable_by(limit, &mut is_less); before.sort_unstable_by(is_less); } ``` So I abandon to use of custom partial_sort. This could fix the issue and risk we are arguing about. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org