-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
Update to grpc-1.47 (and fix the connection-string format) #14125
Conversation
8149ed0
to
6214e2d
Compare
@@ -9,7 +9,7 @@ require ( | |||
github.com/grpc-ecosystem/grpc-gateway v1.16.0 | |||
github.com/stretchr/testify v1.7.0 | |||
google.golang.org/genproto v0.0.0-20211118181313-81c1377c94b1 | |||
google.golang.org/grpc v1.41.0 | |||
google.golang.org/grpc v1.47.0 |
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.
The title mentions 1.48, but actually 1.47
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 guess it's a typo in the PR title as the latest stable release is 1.47. lgmt.
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.
Good catch. Reworded.
624c7bf
to
24886f7
Compare
@ptabor Overall the PR looks good to me. But I don't understand why do you add one more directory |
Signed-off-by: Piotr Tabor <[email protected]>
Signed-off-by: Piotr Tabor <[email protected]>
24886f7
to
cbf54a0
Compare
Removed it. It was produces by intelij. I've just added to it to |
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.
LGTM
…he connection-string format) Signed-off-by: Chao Chen <[email protected]>
…he connection-string format) Signed-off-by: Chao Chen <[email protected]>
…he connection-string format) Signed-off-by: Chao Chen <[email protected]>
…he connection-string format) Signed-off-by: Chao Chen <[email protected]>
Minimal change required to upgrade to grpc-1.47.
#14112. Connecting to
unix:localhost:port
instead of never officially supportedunix://localhost:port
(https://grpc.github.io/grpc/cpp/md_doc_naming.html)