dstandish commented on code in PR #38992: URL: https://github.com/apache/airflow/pull/38992#discussion_r1612108927
########## airflow/models/taskinstance.py: ########## @@ -504,6 +791,34 @@ def _execute_callable(context: Context, **execute_callable_kwargs): return result +def _set_ti_attrs(target, source): + # Fields ordered per model definition + target.start_date = source.start_date + target.end_date = source.end_date + target.duration = source.duration + target.state = source.state + target.try_number = source.try_number + target.max_tries = source.max_tries + target.hostname = source.hostname + target.unixname = source.unixname + target.job_id = source.job_id + target.pool = source.pool + target.pool_slots = source.pool_slots or 1 + target.queue = source.queue + target.priority_weight = source.priority_weight + target.operator = source.operator + target.custom_operator_name = source.custom_operator_name + target.queued_dttm = source.queued_dttm + target.queued_by_job_id = source.queued_by_job_id + target.pid = source.pid + target.executor = source.executor + target.executor_config = source.executor_config + target.external_executor_id = source.external_executor_id + target.trigger_id = source.trigger_id + target.next_method = source.next_method + target.next_kwargs = source.next_kwargs Review Comment: thanks jens -- my goal was to not change the behavior -- i don't know why those fields were not included before. but there could be a reason 🤷 -- 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