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

Downgrade Protobuf from 4.x to 3.x #165

Merged
merged 2 commits into from
Mar 26, 2024
Merged

Downgrade Protobuf from 4.x to 3.x #165

merged 2 commits into from
Mar 26, 2024

Conversation

guilgaly
Copy link
Member

grpc-java is not currently fully compatible with Protobuf 4; see: grpc/grpc-java#11015

In particular, protobuf 4 breaks our demo server due to an incompatibility between the generated code and what the grpc-java library expects.

Motivation:

- grpc-java is not fully compatible with Protobuf 4; see: grpc/grpc-java#11015
- Protobuf 4 breaks our demo server due to an incompatibility between the generated code and what the grpc-java library expects.

Modifications:

- Downgrade to Protobuf 3 on all projects.
- Prevent upgrade to Protobuf 4 in dependabot.yml (note: scala-steward doesn't seem to pick up Protobuf updates anyway, probably because it's not configured as a library dependency per se).

Result:

Stick to Protobuf 3 for now.
Motivation:

We require Java 11 minimum
@guilgaly guilgaly requested a review from slandelle March 26, 2024 08:06
@guilgaly guilgaly self-assigned this Mar 26, 2024
@guilgaly guilgaly merged commit 712d7d0 into main Mar 26, 2024
@guilgaly guilgaly deleted the downgrade-protobuf-3 branch March 26, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants