airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xiaodong DENG (JIRA)" <j...@apache.org>
Subject [jira] [Created] (AIRFLOW-2921) A minor incorrectness in CeleryExecutor()
Date Mon, 20 Aug 2018 09:33:00 GMT
Xiaodong DENG created AIRFLOW-2921:
--------------------------------------

             Summary: A minor incorrectness in CeleryExecutor()
                 Key: AIRFLOW-2921
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-2921
             Project: Apache Airflow
          Issue Type: Improvement
            Reporter: Xiaodong DENG
            Assignee: Xiaodong DENG


Code: [https://github.com/apache/incubator-airflow/blob/62bd68999bade4261daf0499b56098d2e6fc9470/airflow/executors/celery_executor.py#L108]

The logic here is: if a task state becomes either SUCCESS or FAILURE or REVOKED, it will
be removed from *self.tasks()* and *self.last_state()*. However, because line108 is not
indented properly, this task will be added back to *self.last_state()* again.

This will not lead to any error. But may still be worth changing.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message