hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Miklos Szegedi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-5575) Many classes use bare yarn. properties instead of the defined constants
Date Tue, 27 Sep 2016 02:48:21 GMT

    [ 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: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org


Mime
View raw message