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.

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

Switch to gthreads #6111

Open
cnlucas opened this issue Jan 23, 2025 · 0 comments · May be fixed by #6112
Open

Switch to gthreads #6111

cnlucas opened this issue Jan 23, 2025 · 0 comments · May be fixed by #6112
Assignees
@cnlucas

Comments

@cnlucas
Copy link
Member

cnlucas commented Jan 23, 2025

After our gunicorn/gevent research, we found that some of our libraries (including our dbdriver) are not compatible with our current gunicorn gevent worker type. After load testing and several discussions with the team we have decided to switch to threads as we are not able to upgrade and monkeypatch all of our libraries right now.

We can consider gevent workers in the future if threads are not meeting our performance requirements and we are able to upgrade/MP all third party libraries.
@cnlucas cnlucas self-assigned this Jan 23, 2025
@cnlucas cnlucas linked a pull request Jan 23, 2025 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cnlucas cnlucas

Labels
None yet
Projects
Website project
Status: No status
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[DON'T MERGE YET] 6111- switch to gthreads fecgov/openFEC
1 participant
@cnlucas