-
Notifications
You must be signed in to change notification settings - Fork 23
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
Use multiple threads for grpc requests #14
Comments
We'd benefit most from this by having something along #69 |
This crate here, by quickly glancing through the code, starts up a single |
See #37 |
This will require looking at where things are today first... These APIs changed a bit since, but we need to get a better handle on this and certainly prioritize the gRPC workload over the REST APIs one |
The server uses several threads when attending HTTP requests, but only uses 1 when attending grpc requests. I have a WIP branch that fixes this, but it has another problem: both actix and tonic create N threads where N = number of cpus. Tonic seems to use only the threads it creates, but Actix uses also the ones created by Tonic. We'd need to either create just N threads in total and make both frameworks use them or make Actix use only the threads it creates. This only happens when using the async redis storage implementation.
WIP branch: https://github.com/3scale-labs/limitador/tree/run-grpc-reqs-in-multiple-threads
The text was updated successfully, but these errors were encountered: