default min delay is 100ms, not 500ms

Author: jimmyjames

src/main/java/dev/openfga/sdk/util/ExponentialBackoff.java

@@ -19,13 +19,13 @@
  * Utility class for calculating exponential backoff delays with jitter.
  *
  * Implements the retry strategy specified in GitHub issue #155:
- * - Base delay: 2^retryCount * 500ms
+ * - Base delay: 2^retryCount * 100ms
  * - Jitter: Random value between base and 2 * base
- * - Maximum delay: 120 seconds (capped after 8th retry)
+ * - Maximum delay: 120 seconds (capped after 10th retry)
  */
 public class ExponentialBackoff {
 
-    private static final int BASE_DELAY_MS = 500;
+    private static final int BASE_DELAY_MS = 100;
     private static final int MAX_DELAY_SECONDS = 120;
     private static final Random RANDOM = new Random();
 

src/test/java/dev/openfga/sdk/util/ExponentialBackoffTest.java

@@ -30,9 +30,9 @@ void calculateDelay_withRetryCountZero_shouldReturnBaseDelay() {
         Duration result = ExponentialBackoff.calculateDelay(retryCount, fixedRandom);
 
         // Then
-        // For retry count 0: 2^0 * 500ms = 500ms base
-        // With jitter: between 500ms and 1000ms
-        assertThat(result.toMillis()).isBetween(500L, 1000L);
+        // For retry count 0: 2^0 * 100ms = 100ms base
+        // With jitter: between 100ms and 200ms
+        assertThat(result.toMillis()).isBetween(100L, 200L);
     }
 
     @Test
@@ -45,9 +45,9 @@ void calculateDelay_withRetryCountOne_shouldReturnDoubledDelay() {
         Duration result = ExponentialBackoff.calculateDelay(retryCount, fixedRandom);
 
         // Then
-        // For retry count 1: 2^1 * 500ms = 1000ms base
-        // With jitter: between 1000ms and 2000ms
-        assertThat(result.toMillis()).isBetween(1000L, 2000L);
+        // For retry count 1: 2^1 * 100ms = 200ms base
+        // With jitter: between 200ms and 400ms
+        assertThat(result.toMillis()).isBetween(200L, 400L);
     }
 
     @Test
@@ -60,15 +60,15 @@ void calculateDelay_withRetryCountTwo_shouldReturnQuadrupledDelay() {
         Duration result = ExponentialBackoff.calculateDelay(retryCount, fixedRandom);
 
         // Then
-        // For retry count 2: 2^2 * 500ms = 2000ms base
-        // With jitter: between 2000ms and 4000ms
-        assertThat(result.toMillis()).isBetween(2000L, 4000L);
+        // For retry count 2: 2^2 * 100ms = 400ms base
+        // With jitter: between 400ms and 800ms
+        assertThat(result.toMillis()).isBetween(400L, 800L);
     }
 
     @Test
     void calculateDelay_withHighRetryCount_shouldCapAtMaximum() {
         // Given
-        int retryCount = 10; // This would normally result in 2^10 * 500ms = 512000ms
+        int retryCount = 10; // This would normally result in 2^10 * 100ms = 102400ms
         Random fixedRandom = new Random(42); // Fixed seed for deterministic testing
 
         // When
@@ -100,9 +100,9 @@ void calculateDelay_withoutRandomParameter_shouldReturnValidRange() {
         Duration result = ExponentialBackoff.calculateDelay(retryCount);
 
         // Then
-        // For retry count 1: 2^1 * 500ms = 1000ms base
-        // With jitter: between 1000ms and 2000ms
-        assertThat(result.toMillis()).isBetween(1000L, 2000L);
+        // For retry count 1: 2^1 * 100ms = 200ms base
+        // With jitter: between 200ms and 400ms
+        assertThat(result.toMillis()).isBetween(200L, 400L);
     }
 
     @Test
@@ -116,9 +116,9 @@ void calculateDelay_shouldProduceVariousResults() {
         Duration result3 = ExponentialBackoff.calculateDelay(retryCount);
 
         // Then - all should be in valid range but likely different
-        assertThat(result1.toMillis()).isBetween(1000L, 2000L);
-        assertThat(result2.toMillis()).isBetween(1000L, 2000L);
-        assertThat(result3.toMillis()).isBetween(1000L, 2000L);
+        assertThat(result1.toMillis()).isBetween(200L, 400L);
+        assertThat(result2.toMillis()).isBetween(200L, 400L);
+        assertThat(result3.toMillis()).isBetween(200L, 400L);
     }
 
     @Test
@@ -146,7 +146,7 @@ void calculateDelay_progressionTest_shouldFollowExponentialPattern() {
             // Reset the random seed for consistent results across iterations
             fixedRandom.setSeed(42);
             Duration delay = ExponentialBackoff.calculateDelay(i, fixedRandom);
-            long expectedBaseMs = (long) Math.pow(2, i) * 500;
+            long expectedBaseMs = (long) Math.pow(2, i) * 100;
             long expectedMaxMs = Math.min(expectedBaseMs * 2, 120000);
 
             assertThat(delay.toMillis())
@@ -159,7 +159,7 @@ void calculateDelay_progressionTest_shouldFollowExponentialPattern() {
     @Test
     void calculateDelay_atCapThreshold_shouldCapCorrectly() {
         // Given - retry count that would exceed 120s base delay
-        int retryCount = 8; // 2^8 * 500ms = 128000ms > 120000ms
+        int retryCount = 11; // 2^11 * 100ms = 204800ms > 120000ms
         Random fixedRandom = new Random(42);
 
         // When

src/test/java/dev/openfga/sdk/util/RetryStrategyTest.java

@@ -224,8 +224,8 @@ void calculateRetryDelay_withoutRetryAfterHeader_shouldUseExponentialBackoff() {
         Duration result = RetryStrategy.calculateRetryDelay(retryAfterDelay, retryCount);
 
         // Then
-        // For retry count 1: 2^1 * 500ms = 1000ms base, with jitter between 1000ms and 2000ms
-        assertThat(result.toMillis()).isBetween(1000L, 2000L);
+        // For retry count 1: 2^1 * 100ms = 200ms base, with jitter between 200ms and 400ms
+        assertThat(result.toMillis()).isBetween(200L, 400L);
     }
 
     @Test
@@ -238,8 +238,8 @@ void calculateRetryDelay_withEmptyRetryAfterHeader_shouldUseExponentialBackoff()
         Duration result = RetryStrategy.calculateRetryDelay(retryAfterDelay, retryCount);
 
         // Then
-        // For retry count 0: 2^0 * 500ms = 500ms base, with jitter between 500ms and 1000ms
-        assertThat(result.toMillis()).isBetween(500L, 1000L);
+        // For retry count 0: 2^0 * 100ms = 100ms base, with jitter between 100ms and 200ms
+        assertThat(result.toMillis()).isBetween(100L, 200L);
     }
 
     @Test