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.

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

Graceful shutdown timeout #1374

Closed
wants to merge 4 commits into from
Closed

Graceful shutdown timeout #1374

wants to merge 4 commits into from

Conversation

notriddle
Copy link
Contributor

@notriddle notriddle commented Jul 13, 2020

After some feedback, this PR also adds a graceful shutdown wait() function.

@notriddle notriddle changed the title Graceful shutdown Graceful shutdown timeout Jul 13, 2020
@notriddle notriddle force-pushed the async branch 8 times, most recently from 296233e to 15546fa Compare August 16, 2020 00:41
@jebrosen jebrosen requested a review from SergioBenitez August 16, 2020 00:42
@notriddle notriddle force-pushed the async branch 2 times, most recently from e12adfd to c01b864 Compare August 27, 2020 21:05
@notriddle notriddle force-pushed the async branch 12 times, most recently from 1cb09b1 to 3c71840 Compare September 17, 2020 20:51
@notriddle notriddle force-pushed the async branch 4 times, most recently from 003676f to def8225 Compare September 17, 2020 21:30
@SergioBenitez
Copy link
Member

This was discussed on Matrix long ago and we reached the conclusion that the API exposed here is not one we want to move forward with. Thank you for the contribution, however, @notriddle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants