Skip to content
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 grpc version to 1.59.1 and protobuf to 3.24.0 #1008

Merged
merged 5 commits into from
Jun 27, 2024
Merged

Conversation

aman1309
Copy link
Contributor

@aman1309 aman1309 commented Jun 25, 2024

Update grpc version to 1.59.1 and protobuf to 3.24.0
gRPC changes with 1.59.0 https://groups.google.com/g/grpc-io/c/HdrAVplyw7U/m/ki4QQkwOAQAJ
ProtoBuf changes with 3.24.0 https://groups.google.com/g/protobuf/c/mbfAytlUapY/m/n6Ex9uscDAAJ

Tests: tested RC version in dataformats/grpc-infra wc-test.

@@ -1,4 +1,4 @@
version=29.57.0
version=29.57.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the minor version (29.58.0) since this is a significant change.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we're introducing any new feature in pegasus to bump minor version. pegasus features should ideally remain the same. since this is backward compatible major version bump is also not required.

@bohhyang
Copy link
Contributor

Is this tested as a snapshot on some real apps? Please add testing info to the PR description, thanks.

Copy link
Contributor

@bohhyang bohhyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. thx.

@aman1309 aman1309 merged commit 0d3b91c into master Jun 27, 2024
2 checks passed
@aman1309 aman1309 deleted the bumpgrpc branch June 27, 2024 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants