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