[YARN-9547] ContainerStatusPBImpl default execution type is not returned

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.3.0, 3.2.1, 3.1.3
• Assignee: Bilwa S T
• Reporter: Bibin Chundatt
• Created: 13/May/19 11:09
• Updated: 22/Oct/19 13:01
• Resolved: 11/Jun/19 18:24

Description

  @Override
  public synchronized ExecutionType getExecutionType() {
    ContainerStatusProtoOrBuilder p = viaProto ? proto : builder;
    if (!p.hasExecutionType()) {
      return null;
    }
    return convertFromProtoFormat(p.getExecutionType());
  }

ContainerStatusPBImpl executionType should return default as ExecutionType.GUARANTEED.

Issue Links

• is depended upon by: YARN-9301 Too many InvalidStateTransitionException with SLS
• is duplicated by: YARN-9851 Make execution type check compatible