Skip to content

deps: Upgrade grpc-java to 1.83.0 - #13967

Draft
lqiu96 wants to merge 5 commits into
mainfrom
upgrade-grpc-1.83.0
Draft

deps: Upgrade grpc-java to 1.83.0#13967
lqiu96 wants to merge 5 commits into
mainfrom
upgrade-grpc-1.83.0

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 31, 2026

Copy link
Copy Markdown
Member

No description provided.

@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 bumps the version numbers of numerous Google Cloud Java client libraries across their respective README.md files, typically incrementing the minor or patch versions. Additionally, it upgrades the gRPC dependency version from 1.82.2 to 1.83.0 in grpc-gcp-java/pom.xml, librarian.yaml, and sdk-platform-java/gapic-generator-java-pom-parent/pom.xml. I have no feedback to provide as these are standard dependency and documentation version updates.

@lqiu96

lqiu96 commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

Probably blocked by #13976

lqiu96 added a commit that referenced this pull request Aug 6, 2026
…ble writes and read channels (#13976)

Found when upgradeing gRPC to v1.83.0 (as consequently Gson to v2.14.0)
from #13967

Error:
```
18:19:21:843 [ERROR] Errors: 
18:19:21:843 [ERROR]   SerializationTest>BaseSerializationTest.testRestorableObjects:67->BaseSerializationTest.serializeAndDeserialize:82 » JsonSyntax duplicate key: bucket
```

Seems to stem from Gson PR
[#3006](google/gson#3006) which added strict
duplicate key detection in MapTypeAdapterFactory.Adapter.read ('if
(map.containsKey(key)) throw ...').

Possible Fix:
Since com.google.api.services.storage.model.StorageObject extends
com.google.api.client.util.GenericData (which implements java.util.Map),
raw Gson selects MapTypeAdapterFactory when serializing and
deserializing StorageObject. However, GenericData.containsKey returns
true for all @key annotated fields even before they are set, causing raw
Gson.fromJson to throw JsonSyntaxException ('duplicate key') when
deserializing StorageObject.

PR to replaces raw Gson serialization and deserialization of
StorageObject in ApiaryReadRequest and JsonResumableWrite with Google
API Client's official JSON parser/serializer via JsonUtils (backed by
JsonObjectParser / GsonFactory), resolving the duplicate key
deserialization failure in SerializationTest.

Co-authored-by: Dhriti07 <56169283+Dhriti07@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

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.

1 participant