big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

YARN-9683. Remove reapDockerContainerNoPid left behind by YARN-9074 (#1212) Contributed by Kevin Su.

Browse Source 
Reviewed-by: Eric Yang <eyang@apache.org>
Reviewed-by: Adam Antal <adam.antal@cloudera.com>
This commit is contained in:
HUAN-PING SU 2019-08-15 01:42:29 +08:00 committed by Wei-Chiu Chuang
parent c7204417d7
commit 167acd87da
1 changed files with 0 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
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
View File

@ -215,18 +215,4 @@ private boolean sendSignal(String user, String processId,
new ContainerSignalContext.Builder().setContainer(container)
.setUser(user).setPid(processId).setSignal(signal).build());
}
private void reapDockerContainerNoPid(String user) throws IOException {
String containerIdStr =
container.getContainerTokenIdentifier().getContainerID().toString();
LOG.info("Unable to obtain pid, but docker container request detected. "
+ "Attempting to reap container " + containerIdStr);
boolean result = exec.reapContainer(
new ContainerReapContext.Builder()
.setContainer(container)
.setUser(container.getUser())
.build());
LOG.debug("Sent signal to docker container {} as user {}, result={}",
containerIdStr, user, (result ? "success" : "failed"));
}
}