Skip to content

Commit c139406

Browse files
committed
MAPREDUCE-7522. Fix SpotBugs issues in hadoop-mapreduce-client-app module.
1 parent 8c84525 commit c139406

File tree

2 files changed

+4
-1
lines changed
  • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src

2 files changed

+4
-1
lines changed

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/job/impl/TaskAttemptImpl.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2551,4 +2551,7 @@ private void initTaskAttemptStatus(TaskAttemptStatus result) {
25512551
result.counters = counters;
25522552
}
25532553

2554+
public Container getContainer() {
2555+
return container;
2556+
}
25542557
}

hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/v2/app/TestMRApp.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -608,7 +608,7 @@ public void handle(ContainerLauncherEvent event) {
608608
(TaskAttemptImpl) taskAttempts.iterator().next();
609609
// Container from RM should pass through to the launcher. Container object
610610
// should be the same.
611-
assertSame(taskAttempt.container, containerObtainedByContainerLauncher);
611+
assertSame(taskAttempt.getContainer(), containerObtainedByContainerLauncher);
612612
}
613613

614614
private final class MRAppWithHistory extends MRApp {

0 commit comments

Comments
 (0)