darwinyip edited a comment on pull request #16685: URL: https://github.com/apache/airflow/pull/16685#issuecomment-871873235
This flag doesn't seem like a good idea. I agree with the underlying issue this is trying to solve. What about just updating `reattach` directly with this new logic? The flag only has a simple use case in mind with no consideration for concurrency, and that was okay at that time. A more abrupt approach would be forcing every ECS task to take the name `airflow + dag_id + task_id` and have `reattach` check on that. -- 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]
