Skip to content

Add transaction-level isolation level setting #206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

skuruppu
Copy link
Contributor

Enables optionally setting the isolation level at a transaction-level to either SERIALIZABLE or REPEATABLE READ. If the isolation level is not set, Spanner will default to SERIALIZABLE isolation.

Copy link
Collaborator

@yfuruyama yfuruyama left a comment

Choose a reason for hiding this comment

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

Thank you @skuruppu for sending this PR. I left some minor comments, but it's LGTM.

@skuruppu skuruppu force-pushed the isolation-level-support branch from f3fee0c to 82ad132 Compare March 25, 2025 10:32
Enables optionally setting the isolation level at a transaction-level to
either `SERIALIZABLE` or `REPEATABLE READ`. If the isolation level is
not set, Spanner will default to `SERIALIZABLE` isolation.
@skuruppu skuruppu force-pushed the isolation-level-support branch from 82ad132 to 1f2255e Compare March 25, 2025 10:34
@skuruppu
Copy link
Contributor Author

Thanks a lot for the quick review @yfuruyama. I have addressed all comments. I will merge it now.

@skuruppu skuruppu merged commit 687e912 into cloudspannerecosystem:master Mar 25, 2025
2 checks passed
@yfuruyama
Copy link
Collaborator

@skuruppu Thank you for merging that. I tagged it as v0.11.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants