-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Upgrade protobuf-java
to 3.22.2
#10034
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
Comments
To clarify - the problem is solved from your side by updating the version of protobuf, right? From our side - we're currently working on updating the protobuf. We don't know yet if this'd be an easy upgrade, so I can't provide you with a timeline. Ideally it'll be out as a patch-release of 1.54. |
Some others recently brought up the protobuf upgrade, so I made #10041 . But as I mention on it, I think we will hold off for a little bit before merging it to let the ecosystem catch up. Java seems fine, but some of the other languages are having trouble. |
Right, but since we also use
I see, understood and thanks 👍 |
22.3 is out, so we'll target that instead: #10045 . That PR should actually be ready the moment it makes sense to merge. |
Note: users are free to upgrade the version of protobuf in their own dependency tree. The main thing is don't downgrade. Gradle won't do that, and for Maven users we strongly recommend Maven Enforcer's requireUpperBoundDeps. |
#10045 is merged. Let's consider this closed, even if C++ is still a TODO, you wouldn't be able to notice. |
Is your feature request related to a problem?
No
Describe the solution you'd like
Upgrade protobuf from 3.21.7 to 3.22.2 to keep up-to-date with the ecosystem.
Describe alternatives you've considered
n/a
Additional context
The text was updated successfully, but these errors were encountered: