[ https://issues.apache.org/jira/browse/YARN-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13724694#comment-13724694 ]
Vinod Kumar Vavilapalli commented on YARN-966: ---------------------------------------------- bq. after this change are we saying that container's partial localized resources will not be accessible during the state other than LOCALIZED and we will return null?? There is only one user of the API today. We can change it in future. bq. Also I think we are mixing YARN-966 with YARN-906. I don't see any point why we should return null...we can return empty map if no resources are localized.. thoughts? We aren't mixing YARN-966. We are just fixing an assert to be explicitly checked so that IN case something happens, we can report properly. In all my understanding, this code path should never get triggered. > The thread of ContainerLaunch#call will fail without any signal if > getLocalizedResources() is called when the container is not at LOCALIZED > ------------------------------------------------------------------------------------------------------------------------------------------- > > Key: YARN-966 > URL: https://issues.apache.org/jira/browse/YARN-966 > Project: Hadoop YARN > Issue Type: Bug > Reporter: Zhijie Shen > Assignee: Zhijie Shen > Fix For: 2.1.1-beta > > Attachments: YARN-966.1.patch > > > In ContainerImpl.getLocalizedResources(), there's: > {code} > assert ContainerState.LOCALIZED == getContainerState(); // TODO: FIXME!! > {code} > ContainerImpl.getLocalizedResources() is called in ContainerLaunch.call(), > which is scheduled on a separate thread. If the container is not at LOCALIZED > (e.g. it is at KILLING, see YARN-906), an AssertError will be thrown and > fails the thread without notifying NM. Therefore, the container cannot > receive more events, which are supposed to be sent from > ContainerLaunch.call(), and move towards completion. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira