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


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.


> 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

Reply via email to