Github user suez1224 commented on a diff in the pull request:
https://github.com/apache/flink/pull/5416#discussion_r175325243
--- Diff:
flink-yarn-tests/src/test/java/org/apache/flink/yarn/YARNSessionFIFOITCase.java
---
@@ -105,7 +105,7 @@ public void testDetachedMode() throws
InterruptedException, IOException {
}
//additional sleep for the JM/TM to start and establish
connection
- sleep(2000);
+ sleep(3000);
--- End diff --
I found that this test can fail sometimes after the fix, therefore adding 1
second to avoid the false positive. It was not a problem before because this
test used to pass all the time (because it's not doing the right check).
---
