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

Zhijie Shen updated MAPREDUCE-4838:
-----------------------------------

    Attachment: MAPREDUCE-4838_2.patch

Hi Sid,

Thanks for your comments. I've incorporated all your comments in the newest 
patch. In addition, I've added the test cases. Please have a look at the newest 
patch.

One thing which I should point out is that to make the test cases pass, I've 
modified the modifiers of TaskAttemptImpl#dataLocalHosts and 
TaskAttemptImpl#dataLocalRacks from "private final" to "protected". I did this 
because I've reorganized the code, such that these two attributes will only be 
evaluated in the constructor and once. This violates the mock and the spy 
tricks in TestTaskAttempt#testHostResolveAttempt.

I'll create another issue to deal with exposing the additional info through UI.

Thanks,
Zhijie
                
> Add extra info to JH files
> --------------------------
>
>                 Key: MAPREDUCE-4838
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4838
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: Arun C Murthy
>            Assignee: Zhijie Shen
>         Attachments: MAPREDUCE-4838_1.patch, MAPREDUCE-4838_2.patch, 
> MAPREDUCE-4838.patch
>
>
> It will be useful to add more task-info to JH for analytics.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to