Skip to content

Commit 167acd8

Browse files
pingsutwjojochuang
authored andcommitted
YARN-9683. Remove reapDockerContainerNoPid left behind by YARN-9074 (#1212) Contributed by Kevin Su.
Reviewed-by: Eric Yang <[email protected]> Reviewed-by: Adam Antal <[email protected]>
1 parent c720441 commit 167acd8

File tree

1 file changed

+0
-14
lines changed
  • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/launcher

1 file changed

+0
-14
lines changed

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/launcher/ContainerCleanup.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -215,18 +215,4 @@ private boolean sendSignal(String user, String processId,
215215
new ContainerSignalContext.Builder().setContainer(container)
216216
.setUser(user).setPid(processId).setSignal(signal).build());
217217
}
218-
219-
private void reapDockerContainerNoPid(String user) throws IOException {
220-
String containerIdStr =
221-
container.getContainerTokenIdentifier().getContainerID().toString();
222-
LOG.info("Unable to obtain pid, but docker container request detected. "
223-
+ "Attempting to reap container " + containerIdStr);
224-
boolean result = exec.reapContainer(
225-
new ContainerReapContext.Builder()
226-
.setContainer(container)
227-
.setUser(container.getUser())
228-
.build());
229-
LOG.debug("Sent signal to docker container {} as user {}, result={}",
230-
containerIdStr, user, (result ? "success" : "failed"));
231-
}
232218
}

0 commit comments

Comments
 (0)