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

Vinod K V commented on MAPREDUCE-1978:
--------------------------------------

The patch looks good to me too. I'd be happy if the test can be modified to do 
away with all the find/awk kludge - for e.g. {{createHistoryLogsHierarchy()}} 
knows the list of files being created and so can return the same; for the flat 
list, {{File.listFiles()}} should suffice.

The test results were posted a while ago, please run them once more after the 
test changes just to be sure.

> [Rumen] TraceBuilder should provide recursive input folder scanning
> -------------------------------------------------------------------
>
>                 Key: MAPREDUCE-1978
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-1978
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>          Components: tools/rumen
>            Reporter: Amar Kamat
>            Assignee: Ravi Gummadi
>         Attachments: 1978.patch, 1978.v1.patch
>
>
> Currently, {{TraceBuilder}} assumes that the input is either jobhistory files 
> or a folders containing jobhistory files directly underneath the specified 
> folder. There could be a use cases where the input folder could contain 
> sub-folders containing jobhistory files. Rumen should support such input 
> folders.

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