Skip to content
This repository was archived by the owner on Apr 7, 2026. It is now read-only.

fix: keep track of any BeginTransaction option for a Read#1485

Merged
thiagotnunes merged 1 commit intomainfrom
remember-with-transaction-for-read
Oct 6, 2021
Merged

fix: keep track of any BeginTransaction option for a Read#1485
thiagotnunes merged 1 commit intomainfrom
remember-with-transaction-for-read

Conversation

@olavloite
Copy link
Copy Markdown
Collaborator

If a StreamingReadRequest that included a BeginTransaction option was retried as a result
of a transient error (UNAVAILABLE), the fact that the BeginTransaction option was included
would not be registered for the retried request. This could cause a transaction to fail if
the retried request returned an Aborted error, and that Aborted error was caught by the
application.

If a StreamingReadRequest that included a BeginTransaction option was retried as a result
of a transient error (UNAVAILABLE), the fact that the BeginTransaction option was included
would not be registered for the retried request. This could cause a transaction to fail if
the retried request returned an Aborted error, and that Aborted error was caught by the
application.
@olavloite olavloite requested review from a team and thiagotnunes October 5, 2021 13:25
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the googleapis/java-spanner API. label Oct 5, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 5, 2021
@thiagotnunes thiagotnunes merged commit 757d6ec into main Oct 6, 2021
@thiagotnunes thiagotnunes deleted the remember-with-transaction-for-read branch October 6, 2021 00:00
gcf-owl-bot Bot added a commit that referenced this pull request Jul 13, 2022
Source-Link: googleapis/synthtool@7336562
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:d4b80feffe1579818cdc39466152e9de95789a193408506cd4a1ffbe8804dc00
gcf-merge-on-green Bot pushed a commit that referenced this pull request Jul 13, 2022
Source-Link: googleapis/synthtool@7336562
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:d4b80feffe1579818cdc39466152e9de95789a193408506cd4a1ffbe8804dc00
gcf-merge-on-green Bot pushed a commit that referenced this pull request Jul 14, 2022
🤖 I have created a release *beep* *boop*
---


## [6.26.0](v6.25.7...v6.26.0) (2022-07-13)


### Features

* Adding two new fields for Instance create_time and update_time ([#1908](#1908)) ([00b3817](00b3817))
* changes to support data, timestamp and arrays in IT tests ([#1840](#1840)) ([c667653](c667653))
* Error Details Improvement ([c8a2184](c8a2184))
* Error Details Improvement ([#1929](#1929)) ([c8a2184](c8a2184))


### Bug Fixes

* enable longpaths support for windows test ([#1485](#1485)) ([#1946](#1946)) ([fd0b845](fd0b845))


### Dependencies

* update dependency com.google.cloud:google-cloud-trace to v2.3.0 ([#1934](#1934)) ([2813eb2](2813eb2))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
rajatbhatta pushed a commit to rajatbhatta/java-spanner that referenced this pull request Nov 17, 2022
…gleapis#908)

Source-Link: googleapis/synthtool@7336562
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:d4b80feffe1579818cdc39466152e9de95789a193408506cd4a1ffbe8804dc00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: spanner Issues related to the googleapis/java-spanner API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants