GitHub user pepijnve added a comment to the discussion: Should ExecutionPlan 
spawn tasks in `execute` function

I can't give you an authoritative answer on this one, but FWIW 
`CoalescePartitionsExec::execute` also requires a current/active Tokio context 
since it spawns a task for each partition when it's called. In other words, 
it's probably a good idea to have it attached when calling 
`ExecutionPlan::execute`. And `spawn` starts performing work immediately as 
well. So there _is_ one counter example at least.

GitHub link: 
https://github.com/apache/datafusion/discussions/15691#discussioncomment-12925142

----
This is an automatically sent email for github@datafusion.apache.org.
To unsubscribe, please send an email to: 
github-unsubscr...@datafusion.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