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

Ravi Gummadi commented on MAPREDUCE-1865:
-----------------------------------------

Had a look at the code changes othe than the test cases. Changes look good.

One minor comment: Why is the new task status value "SUCCEEDED" changed to 
"SUCCESS" ? Can we do the other way round and changing Values.SUCCESS to 
Values.SUCCEEDED(and changing SUCCESS to SUCCEEDED in JobBuilder when needed, 
because SUCCEEDED is the new value) ?

> [Rumen] Rumen should also support jobhistory files generated using trunk
> ------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1865
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1865
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: tools/rumen
>    Affects Versions: 0.22.0
>            Reporter: Amar Kamat
>            Assignee: Amar Kamat
>             Fix For: 0.22.0
>
>         Attachments: mapreduce-1865-v1.2.patch, mapreduce-1865-v1.6.2.patch
>
>
> Rumen code in trunk parses and process only jobhistory files from pre-21 
> hadoop mapreduce clusters. It should also support jobhistory files generated 
> using trunk.

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