- 
                Notifications
    
You must be signed in to change notification settings  - Fork 13.2k
 
Closed
Labels
good first issueGood for newcomersGood for newcomerskind/enhancementCategory issues or prs related to enhancement.Category issues or prs related to enhancement.
Milestone
Description
There are many users submit issue to describe the GrpcClient has some configuration is too hard to usage, and the client healthy check is too strict.
Refer issues: #8250, #8765, #8771 and other more no response issues.
So I think we can enhance this problem by two actions:
- Make the properties of GrpcClient can be config when client initing. [ISSUE #9013] enhance grpc client #9017
 - Refactor the server check logic to retry several times to avoid disconnect and reconnect frequently. Add configurable for GrpcClient health check retry times #9056
 
Is there some one want to do this enhancement?
After @karsonto finish this two action, I think there also can enhance that abstract all config related logics to one RpcClientConfig and RpcClient only use config from it, and how to get config in RpcClientConfig.
-  Abstract config logic to 
RpcClientConfig 
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomerskind/enhancementCategory issues or prs related to enhancement.Category issues or prs related to enhancement.