-
Notifications
You must be signed in to change notification settings - Fork 277
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
refactor: supervise spawned tasks #4716
Conversation
Like the interface. However i find it a bit hard to track the logic of |
a538a67
to
9dc3b6d
Compare
I agree that the current implementation of Tried to rewrite with less amount of spawns and using Anyway, I found a middle ground and made the implementation simpler. Now Please re-review! |
Indeed, implementation became much simpler and easier to grasp, i like it. |
9dc3b6d
to
0d950ec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
0d950ec
to
f575c53
Compare
f575c53
to
b3f1994
Compare
b3f1994
to
9f5c73b
Compare
9f5c73b
to
699e8da
Compare
Rebased & ran all tests locally - works except 3 extra functional. |
699e8da
to
ae8359d
Compare
ae8359d
to
45e9b7c
Compare
45e9b7c
to
c9ca221
Compare
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
c9ca221
to
0529f36
Compare
Description
This PR introduces a lightweight
Supervisor
for Tokio tasks.What it does:
What it doesn't:
Linked issue
Closes #4698
Related to #4516 (helps to identify failures)
Benefits