[YARN-9883] Reshape SchedulerHealth class

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.0
• Fix Version/s: 3.4.0
• Component/s: resourcemanager, yarn
• Target Version/s: 3.4.0
• Hadoop Flags: Reviewed
• Assignee: D M Murali Krishna Reddy
• Reporter: Adam Antal
• Created: 09/Oct/19 12:24
• Updated: 03/Dec/20 09:08
• Resolved: 03/Dec/20 09:08

Description

The SchedulerHealth class has some flaws, for example:

• It has no javadoc at all
• All its objects are package-private: they should be private
• The internal maps should be (Concurrent) EnumMaps instead of HashMaps: they are more efficient in storing Enums
• schedulerHealthDetails only stores the last operation, its name should reflect that (just like lastSchedulerRunDetails)