wangyang0918 commented on a change in pull request #16623:
URL: https://github.com/apache/flink/pull/16623#discussion_r678796546



##########
File path: 
flink-runtime/src/test/java/org/apache/flink/runtime/resourcemanager/slotmanager/FineGrainedSlotManagerTestBase.java
##########
@@ -142,7 +142,7 @@ static void assertFutureNotComplete(CompletableFuture<?> 
completableFuture) thro
         private final ScheduledExecutor scheduledExecutor = 
TestingUtils.defaultScheduledExecutor();
         private final Executor mainThreadExecutor = MAIN_THREAD_EXECUTOR;
         private FineGrainedSlotManager slotManager;
-        private long requirementCheckDelay = 0;
+        private long requirementCheckDelay = 20;

Review comment:
       We might need to find the root cause instead of simply increasing the 
delay. IIUC, even though the delay is `0`, the test should not fail.




-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

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


Reply via email to