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

Miklos Szegedi commented on YARN-5575:
--------------------------------------

Hi [~templedf],

TestRMApplicationHistoryWriter.java line 392 needs to be split into two lines 
to follow the style of the other branch.

The patch currently looks like:
{code}
    if (isFS) {
      conf.setBoolean(FairSchedulerConfiguration.ASSIGN_MULTIPLE, true);
      conf.set(YarnConfiguration.RM_SCHEDULER, FairScheduler.class.getName());
    } else {
      conf.set(YarnConfiguration.RM_SCHEDULER,
          CapacityScheduler.class.getName());
    }
{code}

> Many classes use bare yarn. properties instead of the defined constants
> -----------------------------------------------------------------------
>
>                 Key: YARN-5575
>                 URL: https://issues.apache.org/jira/browse/YARN-5575
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Daniel Templeton
>            Assignee: Daniel Templeton
>         Attachments: YARN-5575.001.patch
>
>
> MAPREDUCE-5870 introduced the following line:
> {code}
>       conf.setInt("yarn.cluster.max-application-priority", 10);
> {code}
> It should instead be:
> {code}
>       conf.setInt(YarnConfiguration.MAX_CLUSTER_LEVEL_APPLICATION_PRIORITY, 
> 10);
> {code}



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to