Taragolis opened a new pull request, #29447:
URL: https://github.com/apache/airflow/pull/29447

   Right now `EcsRunTaskOperator` when required "reattach" use hacks:
   
   - save info to XCom which not valid because it not use a public interface
   - In additional it can't work with Dynamic Task Mappings (all xcom name 
contain only task_id info and miss other part of unique TI key)
   
   By this PR current mechanism replaced by builtin 
[ECS.Client.run_task](https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/ecs.html#ECS.Client.run_task)
 ability to setup startedBy and filter it later.
   
   `startedBy` limited by 36 characters an it is not possible to use string 
representation of `dag_id` + `task_id` + `run_id` + `map_id`, instead of this 
generate UUID based on this value which can be used as unique (per TI) value.
   
   Unfortunetly right now `EcsRunTaskOperator` set `startedBy` by owner of 
task, so it become mutuality exclusive:
   1. If reattach set to True than `startedBy` set as unique TI (UUID)
   2. If reattach set to False (default) than `startedBy` set as task.owner


-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to