eladkal commented on code in PR #63491:
URL: https://github.com/apache/airflow/pull/63491#discussion_r3503825036
##########
providers/celery/src/airflow/providers/celery/executors/celery_kubernetes_executor.py:
##########
@@ -103,7 +104,10 @@ def _task_event_logs(self, value):
@property
def queued_tasks(self) -> dict[TaskInstanceKey, Any]:
"""Return queued tasks from celery and kubernetes executor."""
- return self.celery_executor.queued_tasks |
self.kubernetes_executor.queued_tasks # type: ignore[return-value]
+ queued_tasks = self.celery_executor.queued_tasks.copy()
Review Comment:
What is the motivation for this change here? If anyone is using
CeleryKubernetesExecutor (Or any of the other static hybrid executors) on
Airflow 3 that is a bigger problem isn't it?
Maybe we should start discussing about removing them before we merge this
change?
--
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]
