YARN-4135. Improve the assertion message in MockRM while failing after waiting for the state.(Nijel S F via rohithsharmaks)

This commit is contained in:
Rohith Sharma K S 2015-09-18 08:44:10 +05:30
parent 6b97fa6652
commit 723c31d45b
2 changed files with 7 additions and 2 deletions

View File

@ -848,6 +848,9 @@ Release 2.8.0 - UNRELEASED
YARN-4078. Add getPendingResourceRequestForAttempt in YarnScheduler interface. YARN-4078. Add getPendingResourceRequestForAttempt in YarnScheduler interface.
(Naganarasimha G R via jianhe) (Naganarasimha G R via jianhe)
YARN-4135. Improve the assertion message in MockRM while failing after waiting for the state.
(Nijel S F via rohithsharmaks)
Release 2.7.2 - UNRELEASED Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

View File

@ -156,7 +156,8 @@ public class MockRM extends ResourceManager {
LOG.info("App State is : " + app.getState()); LOG.info("App State is : " + app.getState());
if (waitedMsecs >= timeoutMsecs) { if (waitedMsecs >= timeoutMsecs) {
Assert.fail("App state is not correct (timedout): expected: " + Assert.fail("App state is not correct (timedout): expected: " +
finalState + " actual: " + app.getState()); finalState + " actual: " + app.getState() +
" for the application " + appId);
} }
} }
@ -185,7 +186,8 @@ public class MockRM extends ResourceManager {
LOG.info("Attempt State is : " + attempt.getAppAttemptState()); LOG.info("Attempt State is : " + attempt.getAppAttemptState());
if (waitedMsecs >= timeoutMsecs) { if (waitedMsecs >= timeoutMsecs) {
Assert.fail("Attempt state is not correct (timedout): expected: " Assert.fail("Attempt state is not correct (timedout): expected: "
+ finalState + " actual: " + attempt.getAppAttemptState()); + finalState + " actual: " + attempt.getAppAttemptState()+
" for the application attempt " + attemptId);
} }
} }