Github user StephanEwen commented on a diff in the pull request:
https://github.com/apache/flink/pull/5656#discussion_r172979459
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/FutureUtils.java
---
@@ -223,6 +224,81 @@
}
}
+ /**
+ * Retry the given operation with the given delay in between successful
completions where the
+ * result does not match a given predicate.
+ *
+ * @param operation to retry
+ * @param retryDelay delay between retries
+ * @param deadline A deadline that specifies at what point we should
stop retrying
+ * @param acceptancePredicate Predicate to test whether the result is
acceptable
+ * @param scheduledExecutor executor to be used for the retry operation
+ * @param <T> type of the result
+ * @return Future which retries the given operation a given amount of
times and delays the retry
+ * in case the predicate isn't matched
+ */
+ public static <T> CompletableFuture<T> retrySuccesfulWithDelay(
+ final Supplier<CompletableFuture<T>> operation,
+ final Time retryDelay,
--- End diff --
Deadline uses `Duration`, this method uses `Time`.
---
