hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "nijel (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-3018) Unify the default value for yarn.scheduler.capacity.node-locality-delay in code and default xml file
Date Wed, 06 May 2015 15:20:00 GMT

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

nijel commented on YARN-3018:
-----------------------------

The test failure can be solved by changing the following lines in TestLeafQueue.testLocalityConstraints()


-    verify(app_0,never()).allocate(eq(NodeType.RACK_LOCAL), eq(node_1_1),  - line number
2394
+    verify(app_0, never()).allocate(eq(NodeType.NODE_LOCAL), eq(node_1_1),  
         any(Priority.class), any(ResourceRequest.class), any(Container.class));
     assertEquals(0, app_0.getSchedulingOpportunities(priority)); 
-    assertEquals(1, app_0.getTotalRequiredResources(priority));        - line number 2397
+    assertEquals(0, app_0.getTotalRequiredResources(priority));

But i am not sure about the impact. Can any one help me in this ? 

> Unify the default value for yarn.scheduler.capacity.node-locality-delay in code and default
xml file
> ----------------------------------------------------------------------------------------------------
>
>                 Key: YARN-3018
>                 URL: https://issues.apache.org/jira/browse/YARN-3018
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: capacityscheduler
>            Reporter: nijel
>            Assignee: nijel
>            Priority: Trivial
>         Attachments: YARN-3018-1.patch, YARN-3018-2.patch, YARN-3018-3.patch, YARN-3018-4.patch
>
>
> For the configuration item "yarn.scheduler.capacity.node-locality-delay" the default
value given in code is "-1"
> public static final int DEFAULT_NODE_LOCALITY_DELAY = -1;
> In the default capacity-scheduler.xml file in the resource manager config directory it
is 40.
> Can it be unified to avoid confusion when the user creates the file without this configuration.
IF he expects the values in the file to be default values, then it will be wrong.



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

Mime
View raw message