-
Notifications
You must be signed in to change notification settings - Fork 5.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DNM] upgrade grpc library to add grpc flow control config #3377
Conversation
@@ -32,6 +32,9 @@ const ( | |||
readTimeoutMedium = 60 * time.Second // For requests that may need scan region. | |||
readTimeoutLong = 150 * time.Second // For requests that may need scan region multiple times. | |||
|
|||
grpcInitialWindowSize = 1 << 30 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reference to use 1GB here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- package: google.golang.org/grpc | ||
version: v1.2.1 | ||
version: 3419b42955675df23457629c75f58eb8dcd56954 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see etcd update grpc-gateway to 1.2.2 with grpc 1.3.x, maybe we can also update it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's not a must, I prefer to take one small step at a time, and leave other upgrade to later PR.
LGTM If etcd upgrades to grpc 1.3, we can do it later. |
@hhkbp2 |
Sure. Let's make two merges. |
Two new PRs will be raised after grpc enters master branch.
This PR gonna be closed. |
Hi,
This PR upgrade grpc library to the version
3419b42955675df23457629c75f58eb8dcd56954
, which has the interfacesto config the flow control in grpc client.
PTAL @nolouch @siddontang @coocood @tiancaiamao @disksing