shunping commented on PR #37536: URL: https://github.com/apache/beam/pull/37536#issuecomment-3922814489
Thanks for contributing to Beam, @shaheeramjad! I went back to check `processes.py` and found the code for error handling there was introduced a few years ago at https://github.com/apache/beam/commit/04db8f58e42a05ea6ebafb331bad041422698def To me, it is very fragile to parse the package name from the argument list, because we can easily make up a example that won't fit. It may be easier to just remove the specific parsing logic for pip and just raise a general traceback. -- 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]
