[YARN-5927] BaseContainerManagerTest::waitForNMContainerState timeout accounting is not accurate

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 2.9.0, 3.0.0-beta1
• Labels: newbie
• Hadoop Flags: Reviewed
• Assignee: Kai
• Reporter: Miklos Szegedi
• Created: 22/Nov/16 19:03
• Updated: 11/Aug/17 19:53
• Resolved: 11/Aug/17 19:41

Description

See below that timeoutSecs is increased twice. We also do a sleep right away before even checking the observed value.

    do {
      Thread.sleep(2000);
 ...
      timeoutSecs += 2;
    } while (!finalStates.contains(currentState)
        && timeoutSecs++ < timeOutMax);