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 2022/06/01 11:56:11 UTC

[GitHub] [airflow] domagojrazum commented on pull request #23870: DAG regex flag in backfill command

domagojrazum commented on PR #23870:
URL: https://github.com/apache/airflow/pull/23870#issuecomment-1143511155

   > There are few more tests will fail if you change "dag" into "dags".
   > 
   > I think this is a bit of compatibility issue to replace it this way - if anyone used that command externally this is a bit unexpected change. I think better approach will be to add separate "dags" field rather than replace "dag" and fail the command if both parameters are specified.
   > 
   > Also it would be great to set some deterministic order of processing the dags. Right now it's a bit random (depends on sequence of hashmap of values in dagbag which depends on sequence of dags returned by SQL query which is not deterministic). Likely sortting the dag_id would be a bit more "stable".
   
   


-- 
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: commits-unsubscribe@airflow.apache.org

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