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

test: Wrap createTestAuthorizedView with retries - #2789

Merged
igorbernstein2 merged 3 commits into
googleapis:mainfrom
trollyxia:it
Feb 24, 2026
Merged

test: Wrap createTestAuthorizedView with retries#2789
igorbernstein2 merged 3 commits into
googleapis:mainfrom
trollyxia:it

Conversation

@trollyxia

Copy link
Copy Markdown
Contributor

Change-Id: I307ff677ffcd007999402b2e55d99226b2be68bc

This is to fix b/477556926.

Change-Id: I307ff677ffcd007999402b2e55d99226b2be68bc
@trollyxia
trollyxia requested a review from a team February 18, 2026 17:03
@trollyxia
trollyxia requested a review from a team as a code owner February 18, 2026 17:03
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: bigtable Issues related to the googleapis/java-bigtable API. labels Feb 18, 2026
@mutianf
mutianf requested a review from a team as a code owner February 18, 2026 22:39
+ tableId
+ ", retryCount: "
+ retryCount);
Thread.sleep(Duration.ofMinutes(1).toMillis());

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

minute seems heavy handed, how about exponential backoff?

// exponential backoff delay starting at 100ms
int expSleep = 100 * Math.pow(2, retryCount);
Thread.sleep(Math.min(expSleep, TimeUnit.MINUTES.toMillis(1));
``

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done!

Change-Id: Ied51f268f8b68f692d97626e7a84a72236901198
@igorbernstein2
igorbernstein2 merged commit 6b2f585 into googleapis:main Feb 24, 2026
24 of 25 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants