Naganarasimha G R commented on MAPREDUCE-6772:

Hi [~devaraj.k],
On rethinking on those two configurations anyway we have disabled it as default 
behavior and if at least it solves the scenario where admin controls MR 
configuration, we can have it. 
Most of the patch looks good to me except for the configuration 
{{"yarn.app.mapreduce.am.container.reuse.enabled"}} can we have it as 
{{"yarn.app.mapreduce.container.reuse.enabled"}}, as with {{".am."}} it sounds 
more like reuse only for am container. 

> Add MR Job Configurations for Containers reuse
> ----------------------------------------------
>                 Key: MAPREDUCE-6772
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6772
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: applicationmaster, mrv2
>            Reporter: Devaraj K
>            Assignee: Devaraj K
>         Attachments: MAPREDUCE-6772-v0.patch, MAPREDUCE-6772-v1.patch
> This task adds configurations required for MR AM Container reuse feature.

This message was sent by Atlassian JIRA

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