Skip to content

Conversation

@ejona86
Copy link
Member

@ejona86 ejona86 commented Jul 28, 2023

The README describes the released version of gRPC. But the most recent release doesn't use protobuf 3.23.4. Telling people to use protoc 3.23.4 is asking for breakages because they will be running with protobuf-java 3.22.3, which is a downgrade.

The bump to 3.23.4 was incorrectly done in #10359.

CC @larry-safran

The README describes the released version of gRPC. But the most recent
release doesn't use protobuf 3.23.4. Telling people to use protoc 3.23.4
is asking for breakages because they will be running with protobuf-java
3.22.3, which is a downgrade.

The bump to 3.23.4 was incorrectly done in grpc#10359.
@ejona86 ejona86 requested a review from sergiitk July 28, 2023 21:47
Copy link
Member

@sergiitk sergiitk left a comment

Choose a reason for hiding this comment

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

Ahh sorry

@sergiitk sergiitk merged commit 4049f89 into grpc:master Aug 3, 2023
@ejona86 ejona86 deleted the readme-no-proto-upgrade branch August 3, 2023 18:25
larry-safran pushed a commit to larry-safran/grpc-java that referenced this pull request Aug 29, 2023
The README describes the released version of gRPC. But the most recent
release doesn't use protobuf 3.23.4. Telling people to use protoc 3.23.4
is asking for breakages because they will be running with protobuf-java
3.22.3, which is a downgrade.

The bump to 3.23.4 was incorrectly done in grpc#10359.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants