Comments (7)
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?
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 noactiveStreams
. Does it violate the original intention of the ping function?
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.
Related Issues (20)
- Minor 'invalid code' error output suggestion
- advancedtls package does not set VerifiedChains in TLSInfo.State HOT 5
- question: Bidirectional Streaming get error: rpc error: code = `Canceled` desc = `context canceled` HOT 6
- Delayed dns resolve after upgrade to v1.60.0 HOT 5
- channelz: SubChannel do not report their Target properly since 1.63 HOT 1
- xds: support LRS for all cluster types HOT 4
- Updating credentials of a running server HOT 5
- clusterimpl: update picker synchronously upon receipt of configuration update HOT 1
- cds: update picker synchronously upon receipt of configuration update
- rls: update picker synchronously upon receipt of configuration update
- Improve grpc-go/Documentation to clearly specify usage of `grpc.SetTrailer` function for adding metadata to send to client
- New ServerOption to set cancel insensitive "values" context. HOT 1
- Return the actual error from unary RPCs when `SendMsg()` returns `io.EOF` HOT 10
- protoc-gen-go-grpc: copy service comment to interfaces HOT 2
- Deprecate Dial and DialContext through a mechanism that doesn't trigger linters HOT 1
- Flaky test: 6/100K: Test/LRSClient HOT 2
- Docs: Document how to migrate from `Dial, DialContext, WithBlock, WithReturnConnectionError, FailOnNonTempDialError` to `NewClient` HOT 3
- Stop supporting 3 releases of Go HOT 2
- <invalid gRPC request method "PRI">, when happend grpc and http use sample port HOT 2
- It seems that there is a memory leak issue in the HandleStreams function. HOT 13
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from grpc-go.