[YARN-11094] Follow up changes for YARN-10547

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.4.0
• Component/s: scheduler-load-simulator
• Hadoop Flags: Reviewed
• Assignee: Szilard Nemeth
• Reporter: Szilard Nemeth
• Created: 22/Mar/22 21:09
• Updated: 25/Mar/22 11:13
• Resolved: 25/Mar/22 11:13

Description

As a follow up of YARN-10547, we need to fix the raw use of Map across the following classes:

• AMDefinitionSLS
• SLSRunner

See this comment for details.

The following methods are affected:

• org.apache.hadoop.yarn.sls.AMDefinitionFactory#createFromSlsTrace
• org.apache.hadoop.yarn.sls.AMDefinitionFactory#getAMContainerResourceSLS
• org.apache.hadoop.yarn.sls.AMDefinitionSLS#getTaskContainers
• Methods of TaskContainerDefinition.Builder
• org.apache.hadoop.yarn.sls.AMDefinitionSLS.Builder#create
• org.apache.hadoop.yarn.sls.SLSRunner#startAMFromSLSTrace

Moreover, some generic cleanup in SLSRunner could be performed to eliminate:
1. Redundant @SuppressWarnings("unchecked") annotations
2. Explicit type arguments for some collections
3. Remove exceptions from methods that never throwing them

Issue Links

• relates to: YARN-10547 Decouple job parsing logic from SLSRunner