feat: make grpc-gcp default enabled #4239

rahul2393 · 2025-11-20T18:47:03Z

This PR enables the gRPC-GCP channel pool extension by default for Cloud Spanner Java client.

What's Changing for Customers

Before this change

gRPC-GCP extension was disabled by default
Default number of channels: 4
Channel pooling was handled by GAX

After this change

gRPC-GCP extension is enabled by default
Default number of channels: 8
Channel pooling is handled by gRPC-GCP extension

Benefits of gRPC-GCP

Improved resilience: When a network connection fails on a particular channel, operations can be automatically retried on a different gRPC channel
Better channel management: gRPC-GCP provides more sophisticated channel affinity and load balancing

How to Disable gRPC-GCP (Switch Back to GAX Channel Pool)

If you need to disable gRPC-GCP and use the previous GAX channel pooling behavior, use the disableGrpcGcpExtension() method:

SpannerOptions options = SpannerOptions.newBuilder()
    .setProjectId("my-project")
    .disableGrpcGcpExtension()
    .build();

When disabled, the default number of channels reverts to 4 (the previous default).

When You Might Want to Disable gRPC-GCP

Maintaining previous behavior: If you want to keep the exact same behavior as before this change (GAX channel pool with 4 default channels).
Troubleshooting: If you experience any unexpected behavior, disabling gRPC-GCP can help isolate whether the issue is related to the channel pooling mechanism.

gemini-code-assist · 2025-11-20T18:47:37Z

Summary of Changes

Hello @rahul2393, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request transitions the grpc-gcp extension in the Spanner client library to be enabled by default. This change simplifies the client's internal logic by removing a configuration flag and its conditional code paths, ensuring that all users benefit from the performance and reliability improvements offered by gRPC-GCP without requiring manual activation.

Highlights

Default gRPC-GCP Enablement: The grpc-gcp extension is now enabled by default, removing the need for explicit configuration to activate it.
Code Simplification: The grpcGcpExtensionEnabled boolean field and all associated conditional logic have been removed from SpannerOptions and GapicSpannerRpc, streamlining the codebase.
Consistent Channel Management: Channel affinity and the number of channels are now consistently managed under the assumption that gRPC-GCP is always active, simplifying channel configuration.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist

Code Review

This pull request makes the grpc-gcp extension enabled by default, which is a good simplification. The changes correctly remove the feature flag and update the related logic. My review includes a few suggestions to improve API clarity and test naming following these changes:

API Clarity: The methods enableGrpcGcpExtension and disableGrpcGcpExtension are now misleading or no-ops. I've suggested addressing this to avoid confusion.
Test Naming: Some test method names have become outdated due to the changes. I've pointed these out with suggestions for new names that better reflect their current purpose.

google-cloud-spanner/src/main/java/com/google/cloud/spanner/SpannerOptions.java

google-cloud-spanner/src/test/java/com/google/cloud/spanner/SpannerOptionsTest.java

sakthivelmanii · 2025-11-21T01:06:16Z

@rahul2393 as a first step, can you please enable it by default without removing the option? If needed, customer can disable it.

google-cloud-spanner-bom/pom.xml

google-cloud-spanner/src/main/java/com/google/cloud/spanner/AbstractReadContext.java

google-cloud-spanner/src/main/java/com/google/cloud/spanner/SpannerOptions.java

olavloite · 2025-12-12T15:41:32Z

google-cloud-spanner/src/test/java/com/google/cloud/spanner/ChannelUsageTest.java

+                          if (call.getMethodDescriptor()
+                              .equals(SpannerGrpc.getBatchCreateSessionsMethod())) {


This should probably be replaced with CreateSession. It works (for now), but once the session pool is removed, this will silently be ignored, as there are no calls to BatchCreateSessions.

Noted, I will update it in separate PR where we are removing session pool

pom.xml

nimf · 2025-12-12T20:43:14Z

Note, that, AFAIU, this will also enable grpc-gcp metrics by default (and these are many more metrics than session metrics), this can be an issue if it's shipped to the custom namespace.

# Conflicts: # google-cloud-spanner/src/main/java/com/google/cloud/spanner/AbstractReadContext.java

rahul2393 · 2025-12-15T07:18:24Z

Thanks @nimf for pointing since Opencensus is already marked deprecated, updated the PR to skip passing MetricRegistry(OpenCensus) to grpc-gcp

* chore: add release-please config for protobuf-4.x (#4249) * fix: Refine connecitivity metrics to capture RPCs with no response he… (#4252) * fix: Refine connecitivity metrics to capture RPCs with no response headers * test fix * fix: retry as PDML dit not retry Resource limit exceeded (#4258) Most transactions that exceed the mutation limit for an atomic transaction will fail with the error "The transaction contains too many mutations.". However, it is also possible that the transaction fails with the more generic error message "Transaction resource limits exceeded". This error did not trigger a retry of the statement using a PDML transaction. Fixes #4253 * deps: update dependency com.google.cloud:sdk-platform-java-config to v3.54.2 (#4261) * deps: update googleapis/sdk-platform-java action to v2.64.2 (#4262) * feat: include RequestID in requests and errors (#4263) - Send a RequestID to Spanner for each request - Make sure that the attempt number of the RequestID is increased if the RPC is retried. - Include the RequestID in every error that is thrown due to an error that is returned by Spanner. * feat: make grpc-gcp default enabled (#4239) This PR enables the gRPC-GCP channel pool extension by default for Cloud Spanner Java client. **What's Changing for Customers** **Before this change** - gRPC-GCP extension was disabled by default - Default number of channels: 4 - Channel pooling was handled by GAX **After this change** - gRPC-GCP extension is enabled by default - Default number of channels: 8 - Channel pooling is handled by gRPC-GCP extension **Benefits of gRPC-GCP** - **Improved resilience:** When a network connection fails on a particular channel, operations can be automatically retried on a different gRPC channel - **Better channel management:** gRPC-GCP provides more sophisticated channel affinity and load balancing **How to Disable gRPC-GCP (Switch Back to GAX Channel Pool)** If you need to disable gRPC-GCP and use the previous GAX channel pooling behavior, use the `disableGrpcGcpExtension()` method: ``` SpannerOptions options = SpannerOptions.newBuilder() .setProjectId("my-project") .disableGrpcGcpExtension() .build(); ``` When disabled, the default number of channels reverts to 4 (the previous default). **When You Might Want to Disable gRPC-GCP** - **Maintaining previous behavior:** If you want to keep the exact same behavior as before this change (GAX channel pool with 4 default channels). - **Troubleshooting**: If you experience any unexpected behavior, disabling gRPC-GCP can help isolate whether the issue is related to the channel pooling mechanism. * chore(main): release 6.104.1-SNAPSHOT (#4248) :robot: I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please). * feat: add support of dynamic channel pooling (#4265) * feat: add support of dynamic channel pooling * set initial pool size to 0 make dynamic channel pool work * chore: generate libraries at Tue Dec 16 06:00:50 UTC 2025 * bump grpc-gcp-java version * fix test * incorporate suggestions * chore: generate libraries at Tue Dec 16 09:56:45 UTC 2025 * make dynamic channel pool default disabled * add verifySameChannelId back and fix partitionDML test * support setting GcpChannelPoolOptions directly --------- Co-authored-by: cloud-java-bot <[email protected]> * chore(main): release 6.105.0 (#4267) * chore(main): release 6.105.0 * chore: generate libraries at Tue Dec 16 17:46:24 UTC 2025 --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: cloud-java-bot <[email protected]> * chore(main): release 6.105.1-SNAPSHOT (#4268) :robot: I have created a release *beep* *boop* --- ### Updating meta-information for bleeding-edge SNAPSHOT release. --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please). * chore: update manifest --------- Co-authored-by: surbhigarg92 <[email protected]> Co-authored-by: Knut Olav Løite <[email protected]> Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: rahul2393 <[email protected]> Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: cloud-java-bot <[email protected]>

rahul2393 requested review from a team as code owners November 20, 2025 18:47

product-auto-label bot added size: m Pull request size is medium. api: spanner Issues related to the googleapis/java-spanner API. labels Nov 20, 2025

gemini-code-assist bot reviewed Nov 20, 2025

View reviewed changes

rahul2393 added do not merge Indicates a pull request not ready for merge, due to either quality or timing. and removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Nov 20, 2025

rahul2393 added 4 commits December 9, 2025 10:10

feat: make grpc-gcp default enabled

89df374

fix test

3498883

add logs

998c8bc

pin grpc-gcp version

c3ef0a5

rahul2393 force-pushed the default-enable-grpc-gcp branch from 4b599e1 to d2c3101 Compare December 9, 2025 04:40

fix test

75ac207

rahul2393 force-pushed the default-enable-grpc-gcp branch from 8bd7007 to 75ac207 Compare December 9, 2025 04:44

rahul2393 added 2 commits December 9, 2025 10:50

fix build

8349844

fix tests

ac8db3d

product-auto-label bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Dec 9, 2025

fix native image tests

eaeef52

rahul2393 force-pushed the default-enable-grpc-gcp branch from cd99542 to eaeef52 Compare December 11, 2025 08:37

rahul2393 and others added 5 commits December 12, 2025 10:22

add ability to disable grpc-gcp if needed

63f0188

fix tests

0380118

Merge branch 'main' into default-enable-grpc-gcp

4cb1192

fix javadoc

bb11190

use channel hint for checking channel usage

044e1f5

olavloite reviewed Dec 12, 2025

View reviewed changes

Merge remote-tracking branch 'origin' into default-enable-grpc-gcp

893d50b

# Conflicts: # google-cloud-spanner/src/main/java/com/google/cloud/spanner/AbstractReadContext.java

rahul2393 added 3 commits December 15, 2025 12:53

fix test

e6704b8

fix test

19eb04b

incorporate changes related to pom.xml

371790e

rahul2393 added the automerge Merge the pull request once unit tests and other checks pass. label Dec 15, 2025

olavloite approved these changes Dec 15, 2025

View reviewed changes

gcf-merge-on-green bot merged commit bb82f9e into main Dec 15, 2025
42 checks passed

gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Dec 15, 2025

gcf-merge-on-green bot deleted the default-enable-grpc-gcp branch December 15, 2025 10:30

release-please bot mentioned this pull request Dec 16, 2025

chore(main): release 6.105.0 #4267

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: make grpc-gcp default enabled #4239

feat: make grpc-gcp default enabled #4239

Uh oh!

rahul2393 commented Nov 20, 2025 •

edited

Loading

Uh oh!

gemini-code-assist bot commented Nov 20, 2025

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sakthivelmanii commented Nov 21, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

olavloite Dec 12, 2025

Uh oh!

rahul2393 Dec 15, 2025

Uh oh!

Uh oh!

nimf commented Dec 12, 2025

Uh oh!

rahul2393 commented Dec 15, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		if (call.getMethodDescriptor()
		.equals(SpannerGrpc.getBatchCreateSessionsMethod())) {

feat: make grpc-gcp default enabled #4239

feat: make grpc-gcp default enabled #4239

Uh oh!

Conversation

rahul2393 commented Nov 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist bot commented Nov 20, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

sakthivelmanii commented Nov 21, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

olavloite Dec 12, 2025

Choose a reason for hiding this comment

Uh oh!

rahul2393 Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nimf commented Dec 12, 2025

Uh oh!

rahul2393 commented Dec 15, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

rahul2393 commented Nov 20, 2025 •

edited

Loading