Skip to content

Revert Handle stuck queued tasks in Celery for db backend#21556

Merged
ephraimbuddy merged 7 commits intoapache:mainfrom
astronomer:test-stuck-task
Apr 2, 2022
Merged

Revert Handle stuck queued tasks in Celery for db backend#21556
ephraimbuddy merged 7 commits intoapache:mainfrom
astronomer:test-stuck-task

Conversation

@ephraimbuddy
Copy link
Contributor

@ephraimbuddy ephraimbuddy commented Feb 14, 2022

This reverts #19769 and #21335


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Feb 14, 2022
@ephraimbuddy ephraimbuddy force-pushed the test-stuck-task branch 2 times, most recently from 0d4db04 to cef1cff Compare February 16, 2022 16:30
@ephraimbuddy ephraimbuddy marked this pull request as ready for review February 18, 2022 10:05
@ephraimbuddy ephraimbuddy changed the title WIP: Fix stuck queued task in Celery Fix stuck queued task in Celery Feb 18, 2022
@ephraimbuddy ephraimbuddy changed the title Fix stuck queued task in Celery WIP: Fix stuck queued task in Celery Feb 18, 2022
@ephraimbuddy ephraimbuddy force-pushed the test-stuck-task branch 2 times, most recently from 85179e7 to 68702be Compare March 20, 2022 08:17
@ephraimbuddy ephraimbuddy changed the title WIP: Fix stuck queued task in Celery Revert Handle stuck queued tasks in Celery for db backend Mar 28, 2022
@ephraimbuddy
Copy link
Contributor Author

Reverting the previous PRs as no one is offering to test and I can't reproduce this locally

@github-actions
Copy link

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Mar 29, 2022
@ephraimbuddy ephraimbuddy force-pushed the test-stuck-task branch 2 times, most recently from 593b6c7 to f8993bb Compare March 30, 2022 06:08
@ephraimbuddy ephraimbuddy reopened this Mar 30, 2022
@ephraimbuddy ephraimbuddy force-pushed the test-stuck-task branch 2 times, most recently from a9f4425 to 9725200 Compare March 31, 2022 12:14
This is an iteration to see if we can fix this issue
@ephraimbuddy ephraimbuddy merged commit 3e0b989 into apache:main Apr 2, 2022
@ephraimbuddy ephraimbuddy deleted the test-stuck-task branch April 2, 2022 10:45
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Scheduler including HA (high availability) scheduler changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants