[ 
https://issues.apache.org/jira/browse/HADOOP-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12664750#action_12664750
 ] 

Hemanth Yamijala commented on HADOOP-5022:
------------------------------------------

I made some very minor changes to the attached patch. The following:

- modified the comments in the code a bit to better reflect the current 
algorithm
- changed the name of the option to retain-master-logs instead of 
retain-masters-logs. Also updated the name in the documentation.
- changed the path being deleted if retain-master-logs is false to remove the 
final / in the patch. This was unnecessary. I've tested this on my local box 
and it seems to work fine.

test-patch results are as follows. Hod tests continue to be done manually, 
outside the unit test cycle of Hadoop.

     [exec] -1 overall.
     [exec]
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec]
     [exec]     -1 tests included.  The patch doesn't appear to include any new 
or modified tests.
     [exec]                         Please justify why no tests are needed for 
this patch.
     [exec]
     [exec]     +1 javadoc.  The javadoc tool did not generate any warning 
messages.
     [exec]
     [exec]     +1 javac.  The applied patch does not increase the total number 
of javac compiler warnings.
     [exec]
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs 
warnings.
     [exec]
     [exec]     +1 Eclipse classpath. The patch retains Eclipse classpath 
integrity.


> [HOD] logcondense should delete all hod logs for a user, including jobtracker 
> logs
> ----------------------------------------------------------------------------------
>
>                 Key: HADOOP-5022
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5022
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: contrib/hod
>            Reporter: Hemanth Yamijala
>            Assignee: Peeyush Bishnoi
>            Priority: Blocker
>             Fix For: 0.18.3
>
>         Attachments: hadoop-5022-1.txt, hadoop-5022-2.txt, hadoop-5022-3.txt, 
> hadoop-5022.txt
>
>
> Currently, logcondense.py does not delete jobtracker logs that it uploads to 
> the DFS when the HOD cluster is deallocated. This will result in the hod-logs 
> directory to slowly accumulate a whole bunch of jobtracker logs. Particularly 
> for users who run a lot of user jobs, this could fill up the namespace.  
> Further these directories will cause the logcondense program to keep 
> repeatedly looking at these directories stressing out the namenode. So, 
> logcondense.py should optionally also delete the jobtracker logs.

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