andreahlert commented on PR #61629: URL: https://github.com/apache/airflow/pull/61629#issuecomment-3886994013
Yeah fair enough, if the SDK is pip-installed normally the transitive deps come with it. The only way it breaks is --no-deps or manual vendoring, which is an edge case I shouldn't be designing around. On the error visibility: yes it does show in task logs as a traceback, but the message isn't great since it's a raw ImportError from deep in the SDK internals. That said, that's a separate issue from what this PR should fix. Reworking the PR based on your feedback. v2 will just restructure the jinja2 template to respect expect_airflow properly and drop the python.py changes entirely. -- 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]
