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 2020/08/25 22:22:23 UTC

[GitHub] [airflow] potiuk commented on pull request #10566: Move dag's cli method out of DAG.

potiuk commented on pull request #10566:
URL: https://github.com/apache/airflow/pull/10566#issuecomment-680298754


   > I wonder if it would be better to delete it. This was helpful when it was easy available, but that leads to a lot of problems. The basic thing is that we have to support running DAGs that have not been loaded by DagBag, so we have to do many things when `import airflow` e.g. add new path to sys.path, loading plugins, etc. If we do not prepare the environment while the airflow import is being done, loading the DAG File may fail.
   
   Maybe as a next move. For now I want to focus on deps.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org