Dandandan commented on pull request #9602:
URL: https://github.com/apache/arrow/pull/9602#issuecomment-792262775


   I think there are some very nice use-cases of this optimization (top-k kind 
of queries), I think it would be very cool to have more advanced features like 
this in Arrow/DataFusion/...
   
   The partial_sort crate seems also quite small (about 60 lines without 
tests/imports). What about having the code in Arrow for now, and (try to) limit 
the amount of unsafety @sundy-li and we can have a next review iteration?


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