Karthik Kambatla commented on YARN-2217:

v8 patch looked mostly good, but for one nit: 
SharedCacheClientImpl#stopClientProxy checks for scmClient not null but doesn't 
set it to null. 

Posted v9 that fixes it. I am +1 on the v9 patch. Will commit it if Jenkins 
doesn't complain of anything. 

> Shared cache client side changes
> --------------------------------
>                 Key: YARN-2217
>                 URL: https://issues.apache.org/jira/browse/YARN-2217
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Chris Trezzo
>            Assignee: Chris Trezzo
>         Attachments: YARN-2217-trunk-v1.patch, YARN-2217-trunk-v2.patch, 
> YARN-2217-trunk-v3.patch, YARN-2217-trunk-v4.patch, YARN-2217-trunk-v5.patch, 
> YARN-2217-trunk-v6.patch, YARN-2217-trunk-v7.patch, YARN-2217-trunk-v8.patch, 
> YARN-2217-trunk-v9.patch
> Implement the client side changes for the shared cache.

This message was sent by Atlassian JIRA

Reply via email to