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

    > Part of the better aggregation spilling (I will file a issue to discuss 
it maybe tonight)
   
   Without having a full understanding of this PR (I have just been following 
the conversation because the change is exciting) my 2ยข is: for us memory 
management is currently one of the biggest thorns with DataFusion. It is quite 
hard at the moment to run with a fixed memory budget given the mix of exceeding 
memory through under accounting and refusing to run operators that can't spill 
/ try to claim more memory than they actually use.


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