-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add Async Lifecycle Handlers #3193
Conversation
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.
I absolutely hate this, but I don't have a better solution and it definitely needs addressing, so here we are 🤦♀️.
Yep been my feelings for a lot of the async work recently! 😅 |
These changes are now available in 4.100.0
Adds new protocol functions to
LifecycleHandler
s to support async contexts. This is important because packages like Redis use this to know when to shutdown their connection pool. In the shutdown function, these call.wait()
which can cause application crashes if called when trying to use NIO's event loop concurrency executor.This provides async alternatives to allow packages to provide full async calls through their stack to avoid these crashes