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

Karthik Kambatla commented on MAPREDUCE-6740:
---------------------------------------------

Latest patch looks good. Couple of minor comments:
# TaskHeartbeatHandler: When declaring taskTimeout, avoid setting the value to 
{{5 * 60 * 1000}}.
# Do we need a test case for when we don't set TASK_REPORT_INTERVAL?

> Enforce mapreduce.task.timeout to be at least 
> mapreduce.task.progress-report.interval
> -------------------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6740
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6740
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: mr-am
>    Affects Versions: 2.8.0
>            Reporter: Haibo Chen
>            Assignee: Haibo Chen
>            Priority: Minor
>         Attachments: mapreduce6740.001.patch, mapreduce6740.002.patch, 
> mapreduce6740.003.patch, mapreduce6740.004.patch, mapreduce6740.005.patch, 
> mapreduce6740.006.patch
>
>
> Mapreduce-6242 makes task status update interval configurable to ease the 
> pressure on MR AM to process status updates, but it did not ensure that 
> mapreduce.task.timeout is no smaller than the configured value of task report 
> interval. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to