AnandInguva commented on PR #17462: URL: https://github.com/apache/beam/pull/17462#issuecomment-1147534253
> Thanks. returning a pipeline from a method called `run_pipeline()`, doesn't sound intuitive, but we could perhaps continue this discussion at a future PR? Depending on what you want to accomplish, there can be other options such as extracting some methods and in the example and then later overriding them in the load test. Yep. For the scope of PR, the refactoring makes sense. I made the change for refactoring. PTAL @tvalentyn -- 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]
