Skip to content

deps: Update Protobuf-Java to v4.33.6 - #13876

Merged
lqiu96 merged 8 commits into
mainfrom
update-protobuf
Jul 27, 2026
Merged

deps: Update Protobuf-Java to v4.33.6#13876
lqiu96 merged 8 commits into
mainfrom
update-protobuf

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 23, 2026

Copy link
Copy Markdown
Member

No description provided.

@lqiu96
lqiu96 requested a review from blakeli0 July 23, 2026 17:04
@lqiu96
lqiu96 requested review from a team as code owners July 23, 2026 17:04

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the protobuf.version property in the parent POM file from 4.33.2 to 4.33.6. There are no review comments, and I have no feedback to provide.

<gson.version>2.13.2</gson.version>
<guava.version>33.5.0-jre</guava.version>
<protobuf.version>4.33.2</protobuf.version>
<protobuf.version>4.33.6</protobuf.version>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we upgrade gen code in the same PR as well?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done

@lqiu96
lqiu96 requested review from a team as code owners July 23, 2026 22:02
@lqiu96
lqiu96 requested a review from a team as a code owner July 23, 2026 22:18
Comment thread librarian.yaml
local_path: sdk-platform-java/hermetic_build/library_generation
protoc:
version: "33.2"
version: "33.6"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we run librarian generate --all in the same PR to update the gen code? I think we may have to manually update it and push it to the same branch. cc: @zhumin8 @JoeWang1127.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes.

Also, line 52 needs to update as well.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oh right. Happy to start a separate discussion on this or to create a librarian ticket if that is better, but is it possible to move this generation step to a nightly build so the Java team doesn't need to remember when to invoke librarian commands?

I would love for librarian generate --all to run automatically on every PR, but IIRC there are limitations that prevent that from happening.

@JoeWang1127 JoeWang1127 Jul 24, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'll help regenerate the repo as @lqiu96 has issues with local setup.

Feel feel to file an issue in librarian repo if you have issues running librarian command.

@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 27, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 27, 2026
@lqiu96
lqiu96 enabled auto-merge (squash) July 27, 2026 20:00
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 27, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 27, 2026
@lqiu96
lqiu96 merged commit 5c6478c into main Jul 27, 2026
326 of 338 checks passed
@lqiu96
lqiu96 deleted the update-protobuf branch July 27, 2026 20:27
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.

4 participants