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

Improve max-worker sync-to-async handling. #423

Open

carltongibson opened this issue Jul 6, 2022 · 0 comments

Labels

enhancement exp/beginner

Member

carltongibson commented Jul 6, 2022

#422 ports the old ASGI_THREADS logic from asgiref. (C.f. #319)

Improvements:

Move the initialisation of the default executor into Server.run() (when configuring the loop).
Adjust the test from Added support for ASGI_THREADS max worker limit. #422 to set up the Server instance, so that's put into effect.
Add a --max-workers CLI flag, with tests etc., to apply this.
Deprecate the env var approach.

The text was updated successfully, but these errors were encountered:

carltongibson pinned this issue

carltongibson unpinned this issue

carltongibson mentioned this issue

Added support for ASGI_THREADS max worker limit. #422

Merged

carltongibson added enhancement exp/beginner labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment