Skip to content

Spanner: Handle aborted transactions when using TransactionManager - #5952

Merged
olavloite merged 1 commit into
googleapis:masterfrom
olavloite:spanner-fix-tx-mgr-abort-handling
Aug 1, 2019
Merged

Spanner: Handle aborted transactions when using TransactionManager#5952
olavloite merged 1 commit into
googleapis:masterfrom
olavloite:spanner-fix-tx-mgr-abort-handling

Conversation

@olavloite

Copy link
Copy Markdown
Contributor

Aborted transactions were not handled correctly by the executeUpdate and batchUpdate methods when used in combination with a TransactionManager. If the user would try to reset the transaction for retry after such an abort, the TransactionManager would throw an IllegalStateException,
because it did not know that the transaction had actually aborted.

Fixes#5951

Aborted transactions were not handled correctly by the executeUpdate and
batchUpdate methods when used in combination with a TransactionManager.
If the user would try to reset the transaction for retry after such an
abort, the TransactionManager would throw an IllegalStateException,
because it did not know that the transaction had actually aborted.
Fixesgoogleapis#5951
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 31, 2019
@olavloite
olavloite requested a review from kolea2July 31, 2019 19:14
@olavloiteolavloite added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 31, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 31, 2019
@olavloiteolavloite added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: spanner Issues related to the Spanner API. labels Aug 1, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spannerIssues related to the Spanner API.cla: yesThis human has signed the Contributor License Agreement.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spanner: executeUpdate() and batchUpdate() could cause AbortedException when using TransactionManager

4 participants

@olavloite@kolea2@googlebot@yoshi-kokoro