[YARN-5184] Fix up incompatible changes introduced on ContainerStatus and NodeReport

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.8.0, 2.9.0
• Fix Version/s: 2.8.0, 3.0.0-alpha2
• Component/s: api
• Target Version/s: 2.8.0, 2.9.0, 3.0.0-alpha2
• Hadoop Flags: Reviewed
• Assignee: Sangjin Lee
• Reporter: Karthik Kambatla
• Created: 31/May/16 16:46
• Updated: 06/Dec/16 23:00
• Resolved: 06/Dec/16 22:50

Description

YARN-2882 and YARN-5430 broke compatibility by adding abstract methods to ContainerStatus. Since ContainerStatus is a Public-Stable class, adding abstract methods to this class breaks any extensions.

To fix this, we should add default implementations to these new methods and not leave them as abstract.

Issue Links

• is broken by: YARN-3866 AM-RM protocol changes to support container resizing
• is broken by: YARN-2882 Add an OPPORTUNISTIC ExecutionType
• is broken by: YARN-5430 Return container's ip and host from NM ContainerStatus call
• is broken by: YARN-4293 ResourceUtilization should be a part of yarn node CLI
• is related to: YARN-5130 Mark ContainerStatus and NodeReport as evolving