Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5416#discussion_r176214419
--- Diff:
flink-yarn-tests/src/test/java/org/apache/flink/yarn/YARNSessionFIFOITCase.java
---
@@ -106,8 +106,16 @@ public void testDetachedMode() throws
InterruptedException, IOException {
}
}
- //additional sleep for the JM/TM to start and establish
connection
- sleep(3000);
+ // additional sleep for the JM/TM to start and establish
connection
+ long startTime = System.currentTimeMillis();
+ while (System.currentTimeMillis() - startTime < 10000 &&
--- End diff --
Let's use `System.nanoTime` for the deadline. `nanoTime` is not susceptible
to clock shifts.
---
