xBis7 commented on code in PR #49180:
URL: https://github.com/apache/airflow/pull/49180#discussion_r2063539044
##########
airflow-core/src/airflow/executors/workloads.py:
##########
@@ -69,7 +69,6 @@ class TaskInstance(BaseModel):
parent_context_carrier: dict | None = None
context_carrier: dict | None = None
- queued_dttm: datetime | None = None
Review Comment:
No, this is intentional. I added it in the previous PR in order to use it as
the `start_time` for the task span in the `base_executor.py`.
https://github.com/apache/airflow/blob/main/airflow-core/src/airflow/executors/base_executor.py#L446
When `workloads.TaskInstance` is initialized, queue time isn't available and
the field ends up `None`. I removed it because it's redundant.
--
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]
