Will C# sdk support "grpc.keepalive_time_ms" option?

[goldenbull]

grpc/include/grpc/impl/codegen/grpc_types.h

Line 222 in 6a1bf82

#define GRPC_ARG_KEEPALIVE_TIME_MS "grpc.keepalive_time_ms"

and other two options: "grpc.keepalive_timeout_ms" and "grpc.keepalive_permit_without_calls"

[jtattermusch]

All the channel arguments are "supported" in the sense that you can create any channel option by e.g.
new ChannelOption("grpc.keepalive_time_ms", yourValue) and then pass it to channel constructor.

[JamesXiao]

Hi @jtattermusch :
I tried this setting about keep alive, but the connection still get closed after idle for like 30 minutes (should not happen if the keep alive ping have bean properly set), and the disconnect event on channel is not triggered, I can only detect the disconnection till a new request is fired.

What I encountered is very similar to this thread:
https://groups.google.com/forum/#!topic/grpc-io/doa2tBP0HK0

I am not sure if there is already a github issue raised by the original author, so I comment here.

[jtattermusch]

@JamesXiao have you set all of "grpc.keepalive_time_ms", "grpc.keepalive_timeout_ms" and "grpc.keepalive_permit_without_calls"?

[JamesXiao]

yes, it works now, with
GRPC_ARG_KEEPALIVE_PERMIT_WITHOUT_CALLS set to 1
and
GRPC_ARG_HTTP2_MAX_PINGS_WITHOUT_DATA set to 0