dheerajturaga commented on PR #57321:
URL: https://github.com/apache/airflow/pull/57321#issuecomment-3459481214

   @kaxil , thanks for addressing this. Let me give more context of our usecase 
here. 
   In Airflow 2, users have a airflow install in their .venv that they create 
in the dags repository. They then make local edits to the dags and use `airflow 
dags list-import-errors` to test and see if they have bugs. If they wish to run 
the dag end to end, they spin up `airflow standalone` and trigger a full dag 
run.
   
   With above implementation, I have a feeling it may cause some confusion 
depending as the command returns different results depending on the state of 
the local sqllite DB. If they make edits + run standalone + make more edits, 
the list/list-import-errors would only reflect what was captured in the db when 
standalone was run. 
   
   What do you think about splitting this command into 2 CLI commands? `airflow 
dags list` and `airflow dags list-from-db` ?
   that way be preserve the old behavior as well as support new list from db? 


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

Reply via email to