You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/04/15 20:50:12 UTC

[GitHub] [arrow] westonpace opened a new pull request #10064: ARROW-12412: [C++] Add ability to convert an AsyncGenerator to an Iterator by wrapping each run with RunInSerialExecutor

westonpace opened a new pull request #10064:
URL: https://github.com/apache/arrow/pull/10064


   The `ProxyExecutor` feels a bit unfortunate.  It could be avoided by changing `AsyncGenerator<T>` to `std::function<Future<T>(Executor*)>` but that would be a pretty expansive change and I'm not sure this feature justifies it.
   
   I was considering putting `ProxyExecutor` in `async_generator.h` instead of `thread_pool.h` so please let me know if that seems like a more appropriate place for it.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] westonpace closed pull request #10064: ARROW-12412: [C++] Add ability to convert an AsyncGenerator to an Iterator by wrapping each run with RunInSerialExecutor

Posted by GitBox <gi...@apache.org>.

westonpace closed pull request #10064:
URL: https://github.com/apache/arrow/pull/10064


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] github-actions[bot] commented on pull request #10064: ARROW-12412: [C++] Add ability to convert an AsyncGenerator to an Iterator by wrapping each run with RunInSerialExecutor

Posted by GitBox <gi...@apache.org>.

github-actions[bot] commented on pull request #10064:
URL: https://github.com/apache/arrow/pull/10064#issuecomment-820721896


   https://issues.apache.org/jira/browse/ARROW-12412


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org