[ 
https://issues.apache.org/jira/browse/YARN-2794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14195273#comment-14195273
 ] 

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
(v6.3.4#6332)

Reply via email to