You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2018/12/29 02:02:11 UTC
[GitHub] jgao54 edited a comment on issue #4389: [AIRFLOW-3583] Fix
AirflowException import
jgao54 edited a comment on issue #4389: [AIRFLOW-3583] Fix AirflowException import
URL: https://github.com/apache/incubator-airflow/pull/4389#issuecomment-450454429
Both `from airflow import AirflowException` and `from airflow.exceptions import AirflowException` should work as AirflowException is imported in the init file. So this is not needed (but improves readability by beng less confusing). Since there are other modules using `from airflow import AirflowException` as well, thought is maybe convert all of them, or leave as is.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services