Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove Uses of Celery.result #2092

Merged
merged 5 commits into from
Dec 24, 2022
Merged

Remove Uses of Celery.result #2092

merged 5 commits into from
Dec 24, 2022

Conversation

IsaacMilarky
Copy link
Contributor

Description

  • Remove all use of Celery's result module to match the recent changes to Celery's settings in augur to ignore task results
  • Make the task phases make more sense and be scheduled as celery tasks

Signed commits

  • [x ] Yes, I signed my commits.

@IsaacMilarky
Remove all use of celery result
99bd992 
Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>
@IsaacMilarky
Fix phase start
af246d0 
Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>
@IsaacMilarky
comment
04f1a17 
Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>
@IsaacMilarky
fix null error in remove_duplicates_by_unique
896f4bf 
Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>
@IsaacMilarky
fix null error in remove_duplicates_by_unique
e617a99 
Signed-off-by: Isaac Milarsky <imilarsky@gmail.com>
@sgoggins sgoggins merged commit 5239f17 into dev Dec 24, 2022
@IsaacMilarky IsaacMilarky deleted the isaac-dev-remove-celery-result branch January 19, 2023 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@IsaacMilarky @sgoggins