ManManson commented on PR #13804: URL: https://github.com/apache/arrow/pull/13804#issuecomment-1224088783
Force-pushed the branch to address review comments from @westonpace. The diff can be found here: https://github.com/apache/arrow/compare/e91396ccf22eec394f23369255a9fd65be60b274..a6c5d5075b8150cf4ecf6874ad59a0e8497af93c Changelog: * Added a non-virtual `GetFragmentsAsyncImplBase`, which accepts a `arrow::internal::Executor*` which is used as a destination executor for the background generator. * Default implementation of virtual `GetFragmentsAsyncImpl` now calls `GetFragmentsAsyncImplBase` with the default executor `internal::GetCPUThreadPool()`. * Expanded the comments section for the default impl method. -- 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]
