Jian He commented on YARN-2212:

- I meant change the stability of new API instead of ApplicationReport
- Previous constructor already set this ?
    if (token.getService() != null) {
      amrmToken.setService(new Text(token.getService()));
- A lot of duplicated code is copied from TestAMRMClient.java, I think we can 
just put the tests there.
- please remove unnecessary testing code
      //setting an instance NMTokenCache
      amClient.setNMTokenCache(new NMTokenCache());
      //asserting we are not using the singleton instance cache
- Please try on secure cluster too.
- test case: test the token is indeed invalided after it expired
    // can do allocate call with new AMRMToken

> ApplicationMaster needs to find a way to update the AMRMToken periodically
> --------------------------------------------------------------------------
>                 Key: YARN-2212
>                 URL: https://issues.apache.org/jira/browse/YARN-2212
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Xuan Gong
>            Assignee: Xuan Gong
>         Attachments: YARN-2212.1.patch, YARN-2212.2.patch, 
> YARN-2212.3.1.patch, YARN-2212.3.patch, YARN-2212.4.patch, YARN-2212.5.patch, 
> YARN-2212.5.patch, YARN-2212.5.rebase.patch

This message was sent by Atlassian JIRA

Reply via email to