westonpace commented on a change in pull request #9643: URL: https://github.com/apache/arrow/pull/9643#discussion_r589783197
########## File path: cpp/src/arrow/util/async_generator.h ########## @@ -177,6 +330,124 @@ class TransformingGenerator { std::shared_ptr<TransformingGeneratorState> state_; }; +/// \brief Transforms an async generator using a transformer function returning a new Review comment: Ah, right. All of these build on each other. So hopefully these changes will magically just go away once #9533 is merged. I think it would be too much work for me to keep track of the branching otherwise. The next PR 11887 needs both these changes and the changes from #9533 so I wouldn't be saving too much by splitting at the moment. ---------------------------------------------------------------- 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: us...@infra.apache.org