[YARN-2869] CapacityScheduler should trim sub queue names when parse configuration

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.7.0
• Component/s: capacityscheduler, resourcemanager
• Target Version/s: 2.7.0
• Hadoop Flags: Reviewed
• Assignee: Wangda Tan
• Reporter: Wangda Tan
• Created: 14/Nov/14 23:25
• Updated: 10/Apr/15 20:10
• Resolved: 06/Dec/14 01:34

Description

Currently, capacity scheduler doesn't trim sub queue name when parsing queue names, for example, the configuration

<configuration>
     <property>
         <name>...root.queues</name>
         <value> a, b  , c</value>
     </property>

     <property>
         <name>...root.b.capacity</name>
         <value>100</value>
     </property>
      
     .......
</property>

Will fail with error:

java.lang.IllegalArgumentException: Illegal capacity of -1.0 for queue root. a 
	at org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.CapacitySchedulerConfiguration.getCapacity(CapacitySchedulerConfiguration.java:332)
	at org.apache.hadoop.yarn.server.resourcemanager.scheduler.capacity.LeafQueue.getCapacityFromConf(LeafQueue.java:196)
        ....

It will try to find a queues with name " a", " b ", and " c", which is apparently wrong, we should do trimming on these sub queue names.