Skip to content

feat(bigtable): throw an error if there is a schema change in metadata returned by ExecuteQueryResponse#15754

Merged
mpeddada1 merged 10 commits intogoogleapis:mainfrom
mpeddada1:schema-change-error
Nov 13, 2025
Merged

feat(bigtable): throw an error if there is a schema change in metadata returned by ExecuteQueryResponse#15754
mpeddada1 merged 10 commits intogoogleapis:mainfrom
mpeddada1:schema-change-error

Conversation

@mpeddada1
Copy link
Collaborator

No description provided.

@product-auto-label product-auto-label bot added the api: bigtable Issues related to the Bigtable API. label Nov 12, 2025
@mpeddada1 mpeddada1 marked this pull request as ready for review November 12, 2025 22:24
@mpeddada1 mpeddada1 requested a review from a team November 12, 2025 22:24
@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

❌ Patch coverage is 92.30769% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.96%. Comparing base (829bce3) to head (b83d689).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...oud/bigtable/internal/data_connection_impl_test.cc 90.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15754      +/-   ##
==========================================
+ Coverage   92.95%   92.96%   +0.01%     
==========================================
  Files        2448     2448              
  Lines      226256   226319      +63     
==========================================
+ Hits       210305   210399      +94     
+ Misses      15951    15920      -31     

☔ 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.

@mpeddada1 mpeddada1 requested a review from scotthart November 12, 2025 23:44
@mpeddada1 mpeddada1 merged commit acd2859 into googleapis:main Nov 13, 2025
68 of 70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants