Skip to content

Commit

Permalink
[Core] fix keepalive in grpc client #33986
Browse files Browse the repository at this point in the history
The keepalive grpc arguments is applied on a channel object that not used, thus it's not affective on worker client. This PR fixed it.
  • Loading branch information
scv119 committed Apr 1, 2023
1 parent 15dfe22 commit 96be4e0
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/ray/rpc/grpc_client.h
Original file line number Diff line number Diff line change
Expand Up @@ -99,9 +99,7 @@ class GrpcClient {
ClientCallManager &call_manager,
bool use_tls = false)
: client_call_manager_(call_manager), use_tls_(use_tls) {
std::shared_ptr<grpc::Channel> channel =
BuildChannel(address, port, CreateDefaultChannelArguments());
channel_ = BuildChannel(address, port);
channel_ = BuildChannel(address, port, CreateDefaultChannelArguments());
stub_ = GrpcService::NewStub(channel_);
}

Expand Down

0 comments on commit 96be4e0

Please sign in to comment.