tvalentyn commented on code in PR #37536: URL: https://github.com/apache/beam/pull/37536#discussion_r2824218187
########## sdks/python/apache_beam/utils/processes.py: ########## @@ -44,6 +44,19 @@ else: + def _pip_package_from_args(args): Review Comment: This is brittle, i'd argue we should print the entire pip command that failed, and its output without attempting to fish out which exact package installation failed. -- 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]
