-
Notifications
You must be signed in to change notification settings - Fork 30
♻️ common public interface for managing long_running_tasks
#7843
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.
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
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7843 +/- ##
==========================================
- Coverage 87.89% 87.27% -0.62%
==========================================
Files 1836 1636 -200
Lines 70992 66370 -4622
Branches 1219 912 -307
==========================================
- Hits 62395 57926 -4469
+ Misses 8254 8182 -72
+ Partials 343 262 -81
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
long_running_tasks
|
What do these changes do?
Aligns the public interface exposed by an
aiohttp
orFastAPI
server for managing the lifecycle of a long running task.This also moves as much of the implementation inside the common
servicelib/long_running_tasks
instead of the individualservicelib/aiohttp/long_running_tasks
andservicelib/fastapi/long_running_tasks
modules.Related issue/s
How to test
Dev-ops