Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/5589#discussion_r171225281 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/concurrent/FutureUtils.java --- @@ -223,6 +223,78 @@ } } + /** + * 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 retries number of retries + * @param retryDelay delay between retries + * @param retryPredicate 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( --- End diff -- the javadocs should say what happens if the number of retries is exhausted
---