[YARN-5776] Checkstyle: MonitoringThread.Run method length is too long

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha2
• Component/s: nodemanager
• Labels: oct16-easy
• Hadoop Flags: Reviewed
• Assignee: Miklos Szegedi
• Reporter: Miklos Szegedi
• Created: 25/Oct/16 01:16
• Updated: 27/May/19 10:06
• Resolved: 27/Oct/16 21:37

Description

YARN-5725 had a check style violation that should be resolved by refactoring the function

Details:
ContainersMonitorImpl.java:395 MonitioringThread.Run @Override:5: Method length is 233 lines (max allowed is 150).

Issue Links

• causes: YARN-9584 Should put initializeProcessTrees method call before get pid
• is duplicated by: YARN-6581 Function ContainersMonitorImpl.MonitoringThread#run() is too long
• relates to: YARN-5725 Test uncaught exception in TestContainersMonitorResourceChange.testContainersResourceChange when setting IP and host