Skip to content

fix(google-cloud-spanner): address compatibility issues with spanner experimental#16593

Merged
olavloite merged 1 commit intogoogleapis:mainfrom
sagnghos:sagnghos/expHost
Apr 10, 2026
Merged

fix(google-cloud-spanner): address compatibility issues with spanner experimental#16593
olavloite merged 1 commit intogoogleapis:mainfrom
sagnghos:sagnghos/expHost

Conversation

@sagnghos
Copy link
Copy Markdown
Contributor

@sagnghos sagnghos commented Apr 9, 2026

Compatibility with experimental host broke post the major refactor in https://github.com/googleapis/python-spanner/pull/1509/changes.

These minor changes fixed the compatibility. Verified with below integration tests run

Integration test successful:

export SPANNER_EXPERIMENTAL_HOST=localhost:15000
python -m pytest -v -s  --disable-warnings tests/system/

@sagnghos sagnghos requested review from a team as code owners April 9, 2026 12:58
Copy link
Copy Markdown
Contributor

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

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 introduces disable_builtin_metrics and _experimental_host to the Spanner clients and updates test configurations. Reviewers identified several issues: _experimental_host is used without being initialized in the constructor, potentially causing an AttributeError, and disable_builtin_metrics is defined within a conditional block, which could lead to an UnboundLocalError. Additionally, the project argument was removed from the AsyncClient instantiation in tests, which is necessary for correct initialization.

@parthea parthea changed the title fix(spanner): address compatibility issues with spanner experimental fix(google-cloud-spanner): address compatibility issues with spanner experimental Apr 9, 2026
Copy link
Copy Markdown
Contributor

@sinhasubham sinhasubham left a comment

Choose a reason for hiding this comment

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

LGTM

@olavloite olavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 10, 2026
@olavloite olavloite merged commit aec1073 into googleapis:main Apr 10, 2026
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kokoro:force-run Add this label to force Kokoro to re-run the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants