[YARN-3018] Unify the default value for yarn.scheduler.capacity.node-locality-delay in code and default xml file

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Component/s: capacityscheduler
• Target Version/s: 2.8.0
• Hadoop Flags: Reviewed
• Assignee: nijel
• Reporter: nijel
• Created: 08/Jan/15 09:29
• Updated: 30/Aug/16 01:28
• Resolved: 08/May/15 21:46

Description

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.