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


Reply via email to