tustvold opened a new pull request, #2203:
URL: https://github.com/apache/arrow-datafusion/pull/2203

   # Which issue does this PR close?
   
   Part of https://github.com/apache/arrow-datafusion/issues/2201.
   
    # Rationale for this change
   
   See ticket
   
   # What changes are included in this PR?
   
   This removes the tokio::spawn from WindowAggExec. I opted to just remove the 
spawn, instead of making it conditional on being run in a tokio runtime, as I'm 
not actually sure why it does this? I guess it might allow for parallelism with 
respect to parent operators, but it seems a little unusual for an operator to 
make this distinction?
   
   # Are there any user-facing changes?
   
   WindowAggExec can be used outside a tokio runtime
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to