[ 
https://issues.apache.org/jira/browse/MAPREDUCE-1186?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Amareshwari Sriramadasu updated MAPREDUCE-1186:
-----------------------------------------------

    Attachment: patch-1186-3.txt

Patch incorporating almost all of review comments, except the following two.

bq. I would like to keep the call for cleanupStorage in 
TaskTracker.initialize(). 
Currently, delete for the local directories happens in two places in 
initialize(). I removed the second one which is essentailly a no-op earlier. 
But now it would delete directories after task-controller's setup.

bq.  Remove LOG.info print in DefaultTaskController.initializeDistributedCache. 
Instead have a LOG.warn in the exception handler.
I have retained LOG.info statement, it would be helpful in debugging. 



> 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.22.0
>
>         Attachments: patch-1186-1.txt, patch-1186-2.txt, 
> patch-1186-3-ydist.txt, patch-1186-3-ydist.txt, patch-1186-3.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