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

Amar Kamat commented on MAPREDUCE-981:
--------------------------------------

Few comments
1) Instead of adding UGI as a constructor parameter to RunningJob, why not do 
something like 
{code}
rJob = new RunningJob(jobId);
rjob.ugi = UserGroupInformation.createRemoteUser(tip.getTask().getUser());
{code}

2) It would be nice to have a getUser() method in RunningJob which might 
internally return 'getUGI().getShortUserName()'. Given a UGI, which username to 
use might be confusing.
3) Using UtilsForTests.InlineCleanupQueue, the wait for cleanup can be avoided.

> TaskTracker does not cleans up job directory when LinuxTraskController is 
> used.
> -------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-981
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-981
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>            Reporter: Karam Singh
>            Assignee: Amareshwari Sriramadasu
>         Attachments: patch-981-1.txt, patch-981.txt
>
>
> TaskTracker does not cleans up job directory when LinuxTraskController is 
> used with wrong permissions and setup task of job fails

-- 
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