Question: PING frame about grpc-go HOT 7 CLOSED

There is another question. The purpose of ping is to maintain the heartbeat when the connection is idle. But with kpDormant mechanism, the heartbeat will not be sent when there is no activeStreams. Does it violate the original intention of the ping function?

Related code

from grpc-go.

There is another question. The purpose of ping is to maintain the heartbeat when the connection is idle. But with kpDormant mechanism, the heartbeat will not be sent when there is no activeStreams. Does it violate the original intention of the ping function?

Related code

From my understanding, the keepalive function is mainly for stream rpc, not for unary rpc.

from grpc-go.

From my understanding, the keepalive function is mainly for stream rpc, not for unary rpc.

Keepalive is less likely to be triggered for unary RPCs with quick replies. Keepalive is primarily triggered when there is a long-lived RPC, which will fail if the keepalive check fails and the connection is closed.

Did you have Keepalive parameters being set to something other than default in the benchmark setup above?

from grpc-go.

From my understanding, the keepalive function is mainly for stream rpc, not for unary rpc.

Keepalive is less likely to be triggered for unary RPCs with quick replies. Keepalive is primarily triggered when there is a long-lived RPC, which will fail if the keepalive check fails and the connection is closed.

Did you have Keepalive parameters being set to something other than default in the benchmark setup above?

No, I'm using grpc's default configuration.

from grpc-go.

Could you send up the repo and maybe the wireshark dump posted above?

from grpc-go.

According to my latest learning results, this Ping may be BDPPing 😆 . WINDOW_UPDATE

if server setting:

server := grpc.NewServer(
		grpc.InitialWindowSize(1 << 24),
		grpc.InitialConnWindowSize(1 << 24),
	)

and client setting:

	options := []grpc.DialOption{
		grpc.WithInitialConnWindowSize(1 << 24),
		grpc.WithInitialWindowSize(1 << 24),
	}

then there will be no more

from grpc-go.

Glad this was resolved!

from grpc-go.