Skip to content

Comments

impl(spanner): add support for Read Lock Mode in RW transactions#15235

Merged
diegomarquezp merged 18 commits intogoogleapis:mainfrom
diegomarquezp:spanner-read-lock-mode
Nov 4, 2025
Merged

impl(spanner): add support for Read Lock Mode in RW transactions#15235
diegomarquezp merged 18 commits intogoogleapis:mainfrom
diegomarquezp:spanner-read-lock-mode

Conversation

@diegomarquezp
Copy link
Collaborator

@diegomarquezp diegomarquezp commented Jun 25, 2025

This change is Reviewable

This adds support for Transaction_ReadWriteOptions_ReadLockMode by adding
a constructor of ReadWriteOptions with a special enum, whose values
can be "optimistic", "pessimistic" and "unspecified".
@product-auto-label product-auto-label bot added the api: spanner Issues related to the Spanner API. label Jun 25, 2025
@codecov
Copy link

codecov bot commented Jun 25, 2025

Codecov Report

❌ Patch coverage is 95.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.08%. Comparing base (4ea3d12) to head (cc03101).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
google/cloud/spanner/transaction.cc 78.57% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15235      +/-   ##
==========================================
- Coverage   93.09%   93.08%   -0.02%     
==========================================
  Files        2442     2442              
  Lines      224759   224818      +59     
==========================================
+ Hits       209247   209277      +30     
- Misses      15512    15541      +29     

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

@diegomarquezp
Copy link
Collaborator Author

Pending: address issues pointed by checkers-pr and clang-tidy-pr

@diegomarquezp diegomarquezp marked this pull request as ready for review November 4, 2025 18:49
@diegomarquezp diegomarquezp requested a review from a team as a code owner November 4, 2025 18:49
@diegomarquezp diegomarquezp merged commit 281be01 into googleapis:main Nov 4, 2025
67 of 71 checks passed
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.

3 participants