[GitHub] [pulsar] rdhabalia commented on a change in pull request #10847: [pulsar-broker] Handle multiple topic creation for same topic-name in broker

[GitBox <gi...@apache.org>]

rdhabalia commented on a change in pull request #10847:
URL: https://github.com/apache/pulsar/pull/10847#discussion_r647656487



##########
File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/FutureUtil.java
##########
@@ -162,4 +164,33 @@ public synchronized Throwable fillInStackTrace() {
             return this;
         }
     }
+
+    public static <T> CompletableFuture<T> futureWithDeadline(ScheduledExecutorService executor, Long delay,

Review comment:
       it is an existing method just moved to `FutureUtil` class. `futureWithDeadline` completes future with exception after the time whereas timeout handling something gets trigger once future is completed with exception.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org