westonpace commented on pull request #11210:
URL: https://github.com/apache/arrow/pull/11210#issuecomment-929528783


   FYI, in the new dataset writer node PR, I've renamed `WaitForTaskToFinish` 
to `End` so it's clear that it should only be called once.  I've also added 
`OnFinished` which returns a future and can be accessed early.  So you can call 
`OnFinished` as much as you want (e.g. before you add any tasks even) but you 
only call `End` once.  I think this clears up the confusion.
   
   Check it out here: https://github.com/apache/arrow/pull/11017/files
   
   If you want to use this I can pull out the changes to AsyncTaskGroup and 
submit them as a small PR on their own today.  Otherways, it will probably be 
some ways off before 11017 merges.


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