Skip to content

Spanner: add backoff for `run_in_transaction' when backend does not provide 'RetryInfo' in response.#8461

Merged
tseaver merged 6 commits intogoogleapis:masterfrom
IlyaFaer:spanner_retries_backoff_7303
Jul 8, 2019
Merged

Spanner: add backoff for `run_in_transaction' when backend does not provide 'RetryInfo' in response.#8461
tseaver merged 6 commits intogoogleapis:masterfrom
IlyaFaer:spanner_retries_backoff_7303

Conversation

@IlyaFaer
Copy link
Copy Markdown

@IlyaFaer IlyaFaer commented Jun 21, 2019

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 21, 2019
IlyaFaer
IlyaFaer commented Jun 21, 2019
View reviewed changes
IlyaFaer
IlyaFaer commented Jun 21, 2019
View reviewed changes
@IlyaFaer IlyaFaer marked this pull request as ready for review June 21, 2019 14:42
crwilcox
crwilcox suggested changes Jun 21, 2019
View reviewed changes
IlyaFaer
IlyaFaer commented Jun 24, 2019
View reviewed changes
@yoshi-automation yoshi-automation added the 🚨 This issue needs some love. label Jun 28, 2019
@crwilcox crwilcox added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 1, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 1, 2019
@crwilcox
Copy link
Copy Markdown
Contributor

crwilcox commented Jul 1, 2019

@IlyaFaer this looks good, but coverage is missing for

google/cloud/spanner_v1/session.py                     126      1     40      2    98%   346, 345->346, 349->exit

@IlyaFaer
Copy link
Copy Markdown
Author

IlyaFaer commented Jul 2, 2019

@crwilcox , okey, I've added two more tests for delay helper itself, and my local coverage says, it's 100% now

@crwilcox crwilcox added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 3, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 3, 2019
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Jul 3, 2019
@IlyaFaer
Copy link
Copy Markdown
Author

IlyaFaer commented Jul 4, 2019

@crwilcox, made commit with black reformat. Sorry, I have a habit of writing in single quotes

@AVaksman AVaksman added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 4, 2019
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jul 4, 2019
@tseaver tseaver changed the title Spanner: Python client not backing off on retries when RetryInfo not … Spanner: add backoff for `run_in_transaction' when backend does not provide 'RetryInfo' in response. Jul 8, 2019
@tseaver tseaver merged commit 56358af into googleapis:master Jul 8, 2019
@tseaver tseaver mentioned this pull request Jul 23, 2019
Closed
parthea pushed a commit that referenced this pull request Mar 9, 2026
@tseaver
Add backoff for `run_in_transaction' when backend does not provide 'R…
c902d01 
…etryInfo' in response. (#8461)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@frankyn frankyn Awaiting requested review from frankyn

@tseaver tseaver Awaiting requested review from tseaver

1 more reviewer

@crwilcox crwilcox crwilcox approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@IlyaFaer @crwilcox @tseaver @googlebot @AVaksman @yoshi-automation @yoshi-kokoro