[ https://issues.apache.org/jira/browse/YARN-1879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14163468#comment-14163468 ]
Tsuyoshi OZAWA commented on YARN-1879: -------------------------------------- Thanks Karthik and Jian for your review! {quote} We should probably move the following check in finishApplicationMaster call upfront so that ApplicationDoesNotExistInCacheException is not thrown for already completed apps. {quote} Good catch. OK, I'll move the check before upfront. {quote} 2. We should also probably add a test that makes duplicate requests to the same/different RM and verify the behavior is as expected. {quote} Maybe TestWorkPreservingRMRestart is good place to add tests. I'll try it. > Mark Idempotent/AtMostOnce annotations to ApplicationMasterProtocol for RM > fail over > ------------------------------------------------------------------------------------ > > Key: YARN-1879 > URL: https://issues.apache.org/jira/browse/YARN-1879 > Project: Hadoop YARN > Issue Type: Sub-task > Components: resourcemanager > Reporter: Jian He > Assignee: Tsuyoshi OZAWA > Priority: Critical > Attachments: YARN-1879.1.patch, YARN-1879.1.patch, > YARN-1879.11.patch, YARN-1879.12.patch, YARN-1879.13.patch, > YARN-1879.14.patch, YARN-1879.15.patch, YARN-1879.16.patch, > YARN-1879.17.patch, YARN-1879.18.patch, YARN-1879.19.patch, > YARN-1879.2-wip.patch, YARN-1879.2.patch, YARN-1879.20.patch, > YARN-1879.21.patch, YARN-1879.22.patch, YARN-1879.23.patch, > YARN-1879.23.patch, YARN-1879.3.patch, YARN-1879.4.patch, YARN-1879.5.patch, > YARN-1879.6.patch, YARN-1879.7.patch, YARN-1879.8.patch, YARN-1879.9.patch > > -- This message was sent by Atlassian JIRA (v6.3.4#6332)