YARN-4135. Improve the assertion message in MockRM while failing after waiting for the state.(Nijel S F via rohithsharmaks)
(cherry picked from commit 723c31d45b
)
This commit is contained in:
parent
1f75e951f5
commit
d94c092ad1
|
@ -796,6 +796,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
|
||||||
|
|
|
@ -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);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue