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

Devaraj Das updated HADOOP-5449:
--------------------------------

       Resolution: Fixed
    Fix Version/s:     (was: 0.20.0)
                   0.19.2
     Hadoop Flags: [Reviewed]
           Status: Resolved  (was: Patch Available)

I just committed this. Thanks, Amareshwari!

> Verify if JobHistory.HistoryCleaner works as expected
> -----------------------------------------------------
>
>                 Key: HADOOP-5449
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5449
>             Project: Hadoop Core
>          Issue Type: Bug
>          Components: mapred
>            Reporter: Amar Kamat
>            Assignee: Amareshwari Sriramadasu
>            Priority: Blocker
>             Fix For: 0.19.2
>
>         Attachments: patch-5449.txt
>
>
> Here is the piece of code I doubt
> {code}
> public void run(){
>       if (isRunning){
>         return;
>       }
>       now = System.currentTimeMillis();
>       // clean history only once a day at max
>       if (lastRan ==0 || (now - lastRan) < ONE_DAY_IN_MS){
>         return;
>       }
>       lastRan = now;
> .....
> // main code for cleaning
> }
> {code}
> {{lastRun}} is initialized to 0 and hence HistoryCleaner will never execute 
> the main code. Also a testcase should be written for 
> JobHistory.HistoryCleaner to check if it works as expected. 

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