Skip to content

Remove TaskFail duplicates check#26714

Merged
dstandish merged 3 commits intoapache:mainfrom
astronomer:remove-taskfail-duplicates-check
Sep 28, 2022
Merged

Remove TaskFail duplicates check#26714
dstandish merged 3 commits intoapache:mainfrom
astronomer:remove-taskfail-duplicates-check

Conversation

@dstandish
Copy link
Contributor

This check was added in error. The duplicates are supposed to be there. The task duration and gantt views process TaskFail records with the apparent intention of including durations from non-final failed attempts to the overall duration of the run.

This check was added in error.  The duplicates are supposed to be there.  The task duration
and gantt views process TaskFail records with the apparent intention of including durations
from non-final failed attempts to the overall duration of the run.
@dstandish dstandish marked this pull request as ready for review September 27, 2022 16:52
@dstandish dstandish requested a review from uranusjr September 27, 2022 16:52
@dstandish dstandish added the type:bug-fix Changelog: Bug Fixes label Sep 27, 2022
@dstandish dstandish added this to the Airflow 2.4.2 milestone Sep 27, 2022
@dstandish dstandish merged commit ee21c1b into apache:main Sep 28, 2022
@dstandish dstandish deleted the remove-taskfail-duplicates-check branch September 28, 2022 05:12
jedcunningham pushed a commit to astronomer/airflow that referenced this pull request Oct 4, 2022
This check was added in error.  The duplicates are supposed to be there.  The task duration
and gantt views process TaskFail records with the apparent intention of including durations
from non-final failed attempts to the overall duration of the run.

(cherry picked from commit ee21c1b)
ephraimbuddy pushed a commit that referenced this pull request Oct 18, 2022
This check was added in error.  The duplicates are supposed to be there.  The task duration
and gantt views process TaskFail records with the apparent intention of including durations
from non-final failed attempts to the overall duration of the run.

(cherry picked from commit ee21c1b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants