parity: default retry presets

Author: GradExMachina

src/aws_durable_execution_sdk_python/retries.py

@@ -5,9 +5,9 @@
 from enum import StrEnum
 import random
 import re
-import sys
 from dataclasses import dataclass, field
 from typing import TYPE_CHECKING
+import math
 
 if TYPE_CHECKING:
     from collections.abc import Callable
@@ -66,7 +66,7 @@ def no_retry(cls) -> RetryDecision:
 
 @dataclass
 class RetryStrategyConfig:
-    max_attempts: int = sys.maxsize  # "infinite", practically
+    max_attempts: int = 3  # "infinite", practically
     initial_delay_seconds: int = 5
     max_delay_seconds: int = 300  # 5 minutes
     backoff_rate: Numeric = 2.0
@@ -109,8 +109,9 @@ def retry_strategy(error: Exception, attempts_made: int) -> RetryDecision:
             config.initial_delay_seconds * (config.backoff_rate ** (attempts_made - 1)),
             config.max_delay_seconds,
         )
-        jitter = config.jitter_strategy.compute_jitter(delay)        
-        final_delay = max(1, delay + jitter)
+        delay_with_jitter = delay + config.jitter_strategy.compute_jitter(delay)
+        delay_with_jitter = math.ceil(delay_with_jitter)
+        final_delay = max(1, delay_with_jitter)
 
         return RetryDecision.retry(round(final_delay))
 
@@ -123,17 +124,18 @@ class RetryPresets:
     @classmethod
     def none(cls) -> Callable[[Exception, int], RetryDecision]:
         """No retries."""
-        return create_retry_strategy(RetryStrategyConfig(max_attempts=0))
+        return create_retry_strategy(RetryStrategyConfig(max_attempts=1))
 
     @classmethod
     def default(cls) -> Callable[[Exception, int], RetryDecision]:
         """Default retries, will be used automatically if retryConfig is missing"""
         return create_retry_strategy(
             RetryStrategyConfig(
-                max_attempts=sys.maxsize,
+                max_attempts=6,
                 initial_delay_seconds=5,
                 max_delay_seconds=60,
                 backoff_rate=2,
+                jitter_strategy=JitterStrategy.FULL
             )
         )