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

Amareshwari Sriramadasu commented on MAPREDUCE-1186:
----------------------------------------------------

bq. Move securing the cache file path in the same code path as where 
localization of the cache files happens.
We should do this for the correctness part, as Hemanth pointed out. 

Hemanth, if we are doing the above, current directory structure is still fine, 
right?  We need not introduce task-id in the directory structure. The patch 
will be replacing current FileUtil.chmod call with the call to TaskController.  
Does it make sense?

> While localizing a DistributedCache file, TT sets permissions recursively on 
> the whole base-dir
> -----------------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1186
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1186
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: tasktracker
>    Affects Versions: 0.21.0
>            Reporter: Vinod K V
>            Assignee: Amareshwari Sriramadasu
>             Fix For: 0.21.0
>
>         Attachments: patch-1186-1.txt, patch-1186-3-ydist.txt, 
> patch-1186-3-ydist.txt, patch-1186-ydist.txt, patch-1186-ydist.txt, 
> patch-1186.txt
>
>
> This is a performance problem.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to