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

Jason Lowe commented on MAPREDUCE-6533:
---------------------------------------

Thanks for updating the patch!  Looks good except for something I just noticed 
-- the test is trying to change the permissions of the root directory, and that 
could fall back to a very public place like /tmp.  The test build directory 
should not be manipulated directly.  Looks like there needs to be another 
directory level added if the permissions really need to be modified on the 
visibility directory's parent.

> testDetermineCacheVisibilities of TestClientDistributedCacheManager is broken
> -----------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6533
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6533
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Chang Li
>            Assignee: Chang Li
>         Attachments: MAPREDUCE-6533.2.patch, MAPREDUCE-6533.3.patch, 
> MAPREDUCE-6533.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to