Skip to content

Commit

Permalink
Put retries back the way they were.
Browse files Browse the repository at this point in the history
I temporarily knocked retry durations down to 1 second which turned a
7-minute test runtime to a 30-second test runtime.  This was a temporary
measure which I found helpful to accelerate iteration on failing tests.
  • Loading branch information
jladieu committed Nov 1, 2023
1 parent 376ac31 commit 2e3c262
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,9 @@ public class TpsRetry {

// The retry function starts with INITIAL_WAIT between retries, and doubles that until it
// reaches MAXIMUM_WAIT, after which all retries are MAXIMUM_WAIT apart.
private static final Duration MAXIMUM_WAIT = Duration.ofSeconds(1);
private static final Duration INITIAL_WAIT = Duration.ofSeconds(1);
private static final Duration OPERATION_TIMEOUT = Duration.ofSeconds(1);
private static final Duration MAXIMUM_WAIT = Duration.ofSeconds(30);
private static final Duration INITIAL_WAIT = Duration.ofSeconds(10);
private static final Duration OPERATION_TIMEOUT = Duration.ofSeconds(300);

// Tps calls which timeout will throw ApiExceptions wrapping SocketTimeoutExceptions and will have
// an errorCode 0. This isn't a real HTTP status code, but we can check for it anyway.
Expand Down

0 comments on commit 2e3c262

Please sign in to comment.