[ https://issues.apache.org/jira/browse/YARN-7958?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16380816#comment-16380816 ]
Gour Saha commented on YARN-7958: --------------------------------- [~csingh] , thank you for the patch. The 002 patch looks good. I am testing it right now. Meanwhile, one comment - Can you move the below line to outside the _existingRecords.forEach_ loop - {code:java} ApplicationId appId = ApplicationId.fromString(app.getId()); {code} > ServiceMaster should only wait for recovery of containers with id that match > the current application id > ------------------------------------------------------------------------------------------------------- > > Key: YARN-7958 > URL: https://issues.apache.org/jira/browse/YARN-7958 > Project: Hadoop YARN > Issue Type: Bug > Components: yarn > Reporter: Chandni Singh > Assignee: Chandni Singh > Priority: Critical > Attachments: YARN-7958.001.patch, YARN-7958.002.patch > > -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org