PolicyExecutor should not preExecute on applyAsync when execution is already recorded

Author: jhalterman

src/main/java/dev/failsafe/spi/PolicyExecutor.java

@@ -105,11 +105,13 @@ public Function<AsyncExecutionInternal<R>, CompletableFuture<ExecutionResult<R>>
     FailsafeFuture<R> future) {
 
     return execution -> {
-      ExecutionResult<R> result = preExecute();
-      if (result != null) {
-        // Still need to preExecute when returning an alternative result before making it to the terminal Supplier
-        execution.preExecute();
-        return CompletableFuture.completedFuture(result);
+      if (!execution.isRecorded()) {
+        ExecutionResult<R> result = preExecute();
+        if (result != null) {
+          // Still need to preExecute when returning an alternative result before making it to the terminal Supplier
+          execution.preExecute();
+          return CompletableFuture.completedFuture(result);
+        }
       }
 
       return innerFn.apply(execution).thenCompose(r -> {