Skip to content

Spanner connection timeout - #4253

Closed
charlesliqlogic wants to merge 7 commits into
googleapis:masterfrom
charlesliqlogic:spanner-connection-timeout
Closed

Spanner connection timeout#4253
charlesliqlogic wants to merge 7 commits into
googleapis:masterfrom
charlesliqlogic:spanner-connection-timeout

Conversation

@charlesliqlogic

@charlesliqlogiccharlesliqlogic commented Dec 26, 2018

Copy link
Copy Markdown
Contributor

Work towards #3616

@charlesliqlogic
charlesliqlogic requested a review from a teamDecember 26, 2018 14:50
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 26, 2018
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 26, 2018
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 27, 2018
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 27, 2018
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 28, 2018
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 28, 2018
ExponentialBackOff backOff = newBackOff();
Context context = Context.current();
int attempt = 0;
int maxAttempt = 6;

This comment was marked as spam.

@charlesliqlogic
charlesliqlogicforce-pushed the spanner-connection-timeout branch from 464a46c to d427495CompareJanuary 10, 2019 15:59
@JesseLovelaceJesseLovelace added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 10, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 10, 2019
@charlesliqlogic
charlesliqlogicforce-pushed the spanner-connection-timeout branch from d427495 to 0c3254bCompareJanuary 11, 2019 19:51
@sduskissduskis added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 7, 2019
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 7, 2019
@yoshi-automationyoshi-automation added the 🚨 critical P0 critical issue. Requires immediate fix label Feb 7, 2019
@sduskissduskis added status: blocked Resolving the issue is dependent on other work. and removed 🚨 critical P0 critical issue. Requires immediate fix labels Feb 11, 2019
@JustinBeckwith

Copy link
Copy Markdown
Contributor

@sduskis what is this blocked on?

@sduskis
sduskis requested a review from kolea2February 26, 2019 14:11
@sduskissduskis removed the status: blocked Resolving the issue is dependent on other work. label Feb 26, 2019
@sduskis

Copy link
Copy Markdown
Contributor

@JustinBeckwith, I removed the blocked. @JesseLovelace or @kolea2, who should review this?

@yoshi-automationyoshi-automation added the 🚨 critical P0 critical issue. Requires immediate fix label Feb 26, 2019
@sduskis

Copy link
Copy Markdown
Contributor

I'm going to close this. We'll need a new PR to fix #3616.

@sduskissduskis closed this Feb 26, 2019
chingor13 pushed a commit that referenced this pull request Mar 24, 2026
Most transactions that exceed the mutation limit for an atomic transaction will fail with the error "The transaction contains too many mutations.". However, it is also possible that the transaction fails with the more generic error message "Transaction resource limits exceeded". This error did not trigger a retry of the statement using a PDML transaction.
Fixes#4253
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.🚨 criticalP0 critical issue. Requires immediate fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@charlesliqlogic@JustinBeckwith@sduskis@JesseLovelace@googlebot@yoshi-automation@yoshi-kokoro