-
Notifications
You must be signed in to change notification settings - Fork 8
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
Support Protobuf Java V4+ #32
Comments
Hi @lockhartja 👋 |
Along the same lines, I had to add com.google.protobuf:protobuf-java-util to my build.gradle.kts after the KrotoDC 1.1.1 update. Without it, |
I've looked into the changes and found out that grpc-java hasn't yet upgraded to protobuf-java V4 (latest version uses 3.25.1). |
@mscheong01 tracking the conversation -- are you able to move to the latest grpc-java...which should allow your community to move protobuf 4? |
@lockhartja I've currently identified a problem where the hasOptionalKeyword method has become private, which breaks the code where we access it 🥲. I haven't had the time to attempt a fix lately, but I'll keep this issue updated |
@mscheong01 was just reviewing and wanted to inquire -- does this blocked by an upstream dependency or simply a waiting for your available bandwidth to increase |
Hi @lockhartja |
Seems to be a breaking change with the latest release of com.google.protobuf:protobuf-java, currently this code
breaks to due to a signature change to com.google.protobuf.GeneratedMessage....
here is a link to the change announcement
[Java] The base class for generated messages will be GeneratedMessage, not GeneratedMessageV3.
any plans to version bump the dependencies?
The text was updated successfully, but these errors were encountered: