Rachelint commented on PR #15591:
URL: https://github.com/apache/datafusion/pull/15591#issuecomment-2848571591

   > Given we have past evidence this approach will work I think we could merge 
it before the queries really sped up. My big concern is that we have a plan to 
eventually avoid both single and multi blocked management
   
   The mainly hard point for removing single management is the `sorted 
aggregation`:
   - We need `Emit::First(exactly n)` for `sorted aggregation`
   - Supporting `Emit::First(exactly n)` in blocked management is actually too 
expansive...
   
   Actually I encounter the same problem in #12996 ... I think maybe it is a 
common problem about how to support `sorted aggregation`?
   


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to