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

gRPC Client - Setting up the Number of request to be queued #10164

Closed
banandh opened this issue May 10, 2023 · 2 comments
Closed

gRPC Client - Setting up the Number of request to be queued #10164

banandh opened this issue May 10, 2023 · 2 comments
Labels

Comments

@banandh
Copy link

banandh commented May 10, 2023

We are doing some performance testing and below are the Observations with the test results and Need feedback on this

1.Even if the upstream gRPC service is up and running, client gets "Unreachable error" at times - what is the exact reason. We have configured retries to resolve the issue. But would like to understand what is happening behind it.

  1. We configured , Semaphore to throttle request in client side even with Semaphore count of 10 , still seeing retries so would like to know the client side connection queueing feature where we can limit the connections on the channel instead of using a separate variables
@ejona86
Copy link
Member

ejona86 commented May 18, 2023

You would need to provide the full error message, including any 'caused by' exceptions, for us to tell you what is happening. We need to see the error to have any hope of telling you the cause.

The channel manages connections. Your semaphore is unlikely to change the number of connections used. It sounds like you want to limit the number of concurrent RPCs because it causes your server load problems. I don't think we have any intention of handling that on client-side as it is best done closer to the application, like you are already doing, because the policy can vary heavily and the application frequency needs to be aware of it to limit memory usage.

You may find community interceptors, like https://github.com/Netflix/concurrency-limits , to be useful. GrpcClientLimiterBuilder may be what you need.

@ejona86
Copy link
Member

ejona86 commented May 25, 2023

No response to provide more information, so closing. More information can be provided and then we can reopen.

@ejona86 ejona86 closed this as completed May 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants