chore: rename to withRetry from RetryableOperation

Author: hsilan

sdk/src/main/java/software/amazon/lambda/durable/util/RetryOperationHelper.java

@@ -79,7 +79,7 @@ private RetryOperationHelper() {
      */
     @SuppressWarnings("unchecked")
     public static <T> T retryOperation(
-            DurableContext context, String name, RetryableOperation<T> operation, RetryOperationConfig config) {
+            DurableContext context, String name, WithRetry<T> operation, RetryOperationConfig config) {
         Objects.requireNonNull(context, "context cannot be null");
         Objects.requireNonNull(name, "name cannot be null");
         Objects.requireNonNull(operation, "operation cannot be null");
@@ -102,8 +102,7 @@ public static <T> T retryOperation(
      * @param config retry configuration including the retry strategy
      * @return the operation result
      */
-    public static <T> T retryOperation(
-            DurableContext context, RetryableOperation<T> operation, RetryOperationConfig config) {
+    public static <T> T retryOperation(DurableContext context, WithRetry<T> operation, RetryOperationConfig config) {
         Objects.requireNonNull(context, "context cannot be null");
         Objects.requireNonNull(operation, "operation cannot be null");
         Objects.requireNonNull(config, "config cannot be null");
@@ -119,7 +118,7 @@ public static <T> T retryOperation(
      * are internal SDK control flow signals that must propagate immediately.
      */
     private static <T> T executeRetryLoop(
-            DurableContext context, String name, RetryableOperation<T> operation, RetryOperationConfig config) {
+            DurableContext context, String name, WithRetry<T> operation, RetryOperationConfig config) {
         var attempt = 1;
         while (true) {
             try {

…bda/durable/util/RetryableOperation.java …mazon/lambda/durable/util/WithRetry.javasdk/src/main/java/software/amazon/lambda/durable/util/RetryableOperation.java renamed to sdk/src/main/java/software/amazon/lambda/durable/util/WithRetry.java sdk/src/main/java/software/amazon/lambda/durable/util/RetryableOperation.java renamed to sdk/src/main/java/software/amazon/lambda/durable/util/WithRetry.java

@@ -13,7 +13,7 @@
  * @param <T> the result type
  */
 @FunctionalInterface
-public interface RetryableOperation<T> {
+public interface WithRetry<T> {
 
     /**
      * Executes the durable operation.

sdk/src/test/java/software/amazon/lambda/durable/util/RetryOperationHelperTest.java

@@ -272,7 +272,7 @@ void nullOperation_shouldThrow() {
 
             assertThrows(
                     NullPointerException.class,
-                    () -> RetryOperationHelper.retryOperation(context, (RetryableOperation<String>) null, config));
+                    () -> RetryOperationHelper.retryOperation(context, (WithRetry<String>) null, config));
         }
 
         @Test

…durable/util/RetryableOperationTest.java …n/lambda/durable/util/WithRetryTest.javasdk/src/test/java/software/amazon/lambda/durable/util/RetryableOperationTest.java renamed to sdk/src/test/java/software/amazon/lambda/durable/util/WithRetryTest.java sdk/src/test/java/software/amazon/lambda/durable/util/RetryableOperationTest.java renamed to sdk/src/test/java/software/amazon/lambda/durable/util/WithRetryTest.java

@@ -8,11 +8,11 @@
 import org.junit.jupiter.api.Test;
 import software.amazon.lambda.durable.DurableContext;
 
-class RetryableOperationTest {
+class WithRetryTest {
 
     @Test
     void canBeImplementedAsLambda() {
-        RetryableOperation<String> operation = (ctx, attempt) -> "result-" + attempt;
+        WithRetry<String> operation = (ctx, attempt) -> "result-" + attempt;
         var context = mock(DurableContext.class);
 
         assertEquals("result-1", operation.execute(context, 1));
@@ -22,7 +22,7 @@ void canBeImplementedAsLambda() {
     @Test
     void receivesContextAndAttempt() {
         var context = mock(DurableContext.class);
-        RetryableOperation<String> operation = (ctx, attempt) -> {
+        WithRetry<String> operation = (ctx, attempt) -> {
             assertSame(context, ctx);
             return "attempt-" + attempt;
         };
@@ -33,7 +33,7 @@ void receivesContextAndAttempt() {
 
     @Test
     void canThrowExceptions() {
-        RetryableOperation<String> operation = (ctx, attempt) -> {
+        WithRetry<String> operation = (ctx, attempt) -> {
             throw new RuntimeException("failed on attempt " + attempt);
         };
         var context = mock(DurableContext.class);
@@ -44,7 +44,7 @@ void canThrowExceptions() {
 
     @Test
     void canReturnNull() {
-        RetryableOperation<String> operation = (ctx, attempt) -> null;
+        WithRetry<String> operation = (ctx, attempt) -> null;
         var context = mock(DurableContext.class);
 
         assertNull(operation.execute(context, 1));