potiuk commented on PR #23870: URL: https://github.com/apache/airflow/pull/23870#issuecomment-1141954747
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 ratehr 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
