Skip to content

feat(spanner): [WIP] support data client#4701

Closed
olavloite wants to merge 13 commits intogoogleapis:mainfrom
olavloite:spanner-client
Closed

feat(spanner): [WIP] support data client#4701
olavloite wants to merge 13 commits intogoogleapis:mainfrom
olavloite:spanner-client

Conversation

@olavloite
Copy link
Copy Markdown
Contributor

@olavloite olavloite commented Feb 16, 2026

[WORK IN PROGRESS]

Add a data-plane client for Spanner.

Depends on googleapis/librarian#4072

Add a data-plane client for Spanner.
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the Spanner API. label Feb 16, 2026
@coryan
Copy link
Copy Markdown
Contributor

coryan commented Feb 16, 2026

/gcbrun

Copy link
Copy Markdown
Contributor

@coryan coryan left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I think you can skip the grpc-mock subdirectory in the first PR and things will be easier to review and merge.

Also filed googleapis/librarian#4073 because the grpc-mock thing is tedious. Do not worry, I believe we should continue to make progress with this PR (maybe split in smaller PRs) and leave the sidekick automation for later.

Comment thread src/gax-internal/src/grpc.rs
Comment thread src/spanner/grpc-mock/protos/google/api/apikeys/v2/BUILD.bazel Outdated
Comment thread src/spanner/grpc-mock/protos/google/api/apikeys/v2/apikeys_v2.yaml Outdated
Comment thread src/spanner/grpc-mock/protos/google/api/apikeys/BUILD.bazel Outdated
Comment thread src/spanner/grpc-mock/protos/google/api/expr/conformance/v1alpha1/BUILD.bazel Outdated
Comment thread src/spanner/src/client.rs
Comment thread src/spanner/grpc-mock/protos/google/spanner/admin/instance/v1/BUILD.bazel Outdated
@coryan
Copy link
Copy Markdown
Contributor

coryan commented Feb 16, 2026

/gcbrun

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 0% with 226 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.93%. Comparing base (f9355a3) to head (cbf5788).
⚠️ Report is 176 commits behind head on main.

Files with missing lines Patch % Lines
src/spanner/src/read_write_transaction.rs 0.00% 50 Missing ⚠️
src/spanner/src/statement.rs 0.00% 31 Missing ⚠️
src/spanner/src/read_context.rs 0.00% 30 Missing ⚠️
src/spanner/src/row.rs 0.00% 25 Missing ⚠️
src/spanner/src/mutation.rs 0.00% 22 Missing ⚠️
src/gax-internal/src/grpc.rs 0.00% 21 Missing ⚠️
src/spanner/src/client/builder.rs 0.00% 18 Missing ⚠️
src/spanner/src/partitioned_dml.rs 0.00% 10 Missing ⚠️
src/spanner/grpc-mock/src/lib.rs 0.00% 9 Missing ⚠️
src/spanner/src/client.rs 0.00% 9 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4701      +/-   ##
==========================================
- Coverage   95.02%   89.93%   -5.09%     
==========================================
  Files         204      215      +11     
  Lines        7877     8320     +443     
==========================================
- Hits         7485     7483       -2     
- Misses        392      837     +445     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@olavloite
Copy link
Copy Markdown
Contributor Author

No longer relevant.

@olavloite olavloite closed this Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the Spanner API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants