[ https://issues.apache.org/jira/browse/YARN-2208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14056462#comment-14056462 ]
Xuan Gong commented on YARN-2208: --------------------------------- bq. In which case this if condition can happen? For the test cases. Anyway, I have removed those codes. bq. merge the duplicate code DONE bq. test case: original testMasterKeyRollOver is not enough, it calls rollMasterKey to roll master key. But after this patch, the new master key will be activated only after activation delay. we need to test 1) the original token is still valid after key rolled over but before activated, and 2) invalid after the new master key is activated. 3) new token can be generated off the new master instead of old master key. DONE > AMRMTokenManager need to have a way to roll over AMRMToken > ---------------------------------------------------------- > > Key: YARN-2208 > URL: https://issues.apache.org/jira/browse/YARN-2208 > Project: Hadoop YARN > Issue Type: Sub-task > Components: resourcemanager > Reporter: Xuan Gong > Assignee: Xuan Gong > Attachments: YARN-2208.1.patch, YARN-2208.2.patch, YARN-2208.3.patch, > YARN-2208.4.patch, YARN-2208.5.patch > > -- This message was sent by Atlassian JIRA (v6.2#6252)