[ https://issues.apache.org/jira/browse/YARN-1430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13829400#comment-13829400 ]
Omkar Vinit Joshi commented on YARN-1430: ----------------------------------------- I think for now we should add assert statements so that in test environment it will always fail making sure we are not missing some invalid transitions? YARN-1416 is one of those examples. I agree with [~vinodkv] and [~jlowe]. Probably we should be consistent everywhere and should show somewhere these system critical errors without actually crashing daemons. > InvalidStateTransition exceptions are ignored in state machines > --------------------------------------------------------------- > > Key: YARN-1430 > URL: https://issues.apache.org/jira/browse/YARN-1430 > Project: Hadoop YARN > Issue Type: Bug > Reporter: Omkar Vinit Joshi > Assignee: Omkar Vinit Joshi > > We have all state machines ignoring InvalidStateTransitions. These exceptions > will get logged but will not crash the RM / NM. We definitely should crash it > as they move the system into some invalid / unacceptable state. > * Places where we hide this exception :- > ** JobImpl > ** TaskAttemptImpl > ** TaskImpl > ** NMClientAsyncImpl > ** ApplicationImpl > ** ContainerImpl > ** LocalizedResource > ** RMAppAttemptImpl > ** RMAppImpl > ** RMContainerImpl > ** RMNodeImpl > thoughts? -- This message was sent by Atlassian JIRA (v6.1#6144)