YARN-4171. Fix findbugs warnings in YARN-1197 branch. Contributed by Wangda Tan

This commit is contained in:
Jian He 2015-09-18 16:42:26 +08:00 committed by Wangda Tan
parent 89cab1ba5f
commit b3f6b641dc
3 changed files with 3 additions and 3 deletions

View File

@ -900,6 +900,8 @@ Release 2.8.0 - UNRELEASED
YARN-4188. Make MoveApplicationAcrossQueues abstract, newInstance static. YARN-4188. Make MoveApplicationAcrossQueues abstract, newInstance static.
(Giovanni Matteo Fumarola via cdouglas) (Giovanni Matteo Fumarola via cdouglas)
YARN-4171. Fix findbugs warnings in YARN-1197 branch. (Wangda Tan via jianhe)
Release 2.7.2 - UNRELEASED Release 2.7.2 - UNRELEASED
INCOMPATIBLE CHANGES INCOMPATIBLE CHANGES

View File

@ -93,6 +93,7 @@ public void setLogAggregationReportsForApps(
this.logAggregationReportsForApps = logAggregationReportsForApps; this.logAggregationReportsForApps = logAggregationReportsForApps;
} }
@SuppressWarnings("unchecked")
public List<Container> getNMReportedIncreasedContainers() { public List<Container> getNMReportedIncreasedContainers() {
return nmReportedIncreasedContainers == null ? Collections.EMPTY_LIST return nmReportedIncreasedContainers == null ? Collections.EMPTY_LIST
: nmReportedIncreasedContainers; : nmReportedIncreasedContainers;

View File

@ -478,9 +478,6 @@ private Container updateContainerAndNMToken(RMContainer rmContainer,
ContainerType containerType = ContainerType.TASK; ContainerType containerType = ContainerType.TASK;
// The working knowledge is that masterContainer for AM is null as it // The working knowledge is that masterContainer for AM is null as it
// itself is the master container. // itself is the master container.
RMAppAttempt appAttempt = rmContext.getRMApps()
.get(container.getId().getApplicationAttemptId().getApplicationId())
.getCurrentAppAttempt();
if (isWaitingForAMContainer(getApplicationId())) { if (isWaitingForAMContainer(getApplicationId())) {
containerType = ContainerType.APPLICATION_MASTER; containerType = ContainerType.APPLICATION_MASTER;
} }