Tsuyoshi OZAWA commented on YARN-2794:

[~jianhe], oops, I got it. The updating about systemCredentials is done only 
via setSystemCredentials. Then, your solution is enough. 
One minor nits: 
TestLogAggregationService#testAddNewTokenSentFromRMForLogAggregation calls   
{{this.context.getSystemCredentialsForApps().put(application1, credentials);}}. 
We should use ConcurrentHashMap for the test case.

> Fix log msgs about distributing system-credentials 
> ---------------------------------------------------
>                 Key: YARN-2794
>                 URL: https://issues.apache.org/jira/browse/YARN-2794
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-2794.patch

This message was sent by Atlassian JIRA

Reply via email to