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 2021/07/01 02:59:00 UTC

[GitHub] [airflow] darwinyip edited a comment on pull request #16685: reattach_prev_task ECSOperator

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

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