[YARN-6953] Clean up ResourceUtils.setMinimumAllocationForMandatoryResources() and setMaximumAllocationForMandatoryResources()

• Type: Sub-task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: YARN-3926
• Fix Version/s: 3.1.0, 3.0.1
• Component/s: resourcemanager
• Labels: newbie
• Target Version/s: YARN-3926
• Hadoop Flags: Reviewed
• Assignee: Manikandan R
• Reporter: Daniel Templeton
• Created: 04/Aug/17 19:28
• Updated: 15/Nov/17 19:16
• Resolved: 15/Nov/17 18:14

Description

The setMinimumAllocationForMandatoryResources() and setMaximumAllocationForMandatoryResources() methods are quite convoluted. They'd be much simpler if they just handled CPU and memory manually instead of trying to be clever about doing it in a loop. There are also issues, such as the log warning always talking about memory or the last element of the inner array being a copy of the first element.