alamb commented on PR #16196:
URL: https://github.com/apache/datafusion/pull/16196#issuecomment-2922888436

   I also agree we could make a query cancellation token work -- the potential 
challenges of that approach are:
   1. We have to find some way to trigger the cancellation token when the 
streams are dropped 
   2. We have to update all operators to *also* check their cancellation tokens
   
   I don't see that this is fundamentally different than what this PR does 
(basically has HashAggregateStream "check" for cancellation every 64 batches
   
   
   


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