From 114b064257c2b04f054229f0b20af360ce4e5ada Mon Sep 17 00:00:00 2001 From: Sunil G Date: Wed, 11 Jul 2018 08:54:13 +0530 Subject: [PATCH] Addendum fix for YARN-8473. Containers being launched as app tears down can leave containers in NEW state --- .../nodemanager/containermanager/application/ApplicationImpl.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/ApplicationImpl.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/ApplicationImpl.java index db13dde9c95..d3f6b42258f 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/ApplicationImpl.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/application/ApplicationImpl.java @@ -330,8 +330,8 @@ public void transition(ApplicationImpl app, ApplicationEvent event) { // these get queued up and sent out in AppInitDoneTransition break; default: - LOG.warn("Killing {} because {} is in state {}", - container.getContainerId(), app, appState); + LOG.warn("Killing " + container.getContainerId() + " because " + app + + " is in state " + appState); app.dispatcher.getEventHandler().handle(new ContainerKillEvent( container.getContainerId(), ContainerExitStatus.KILLED_AFTER_APP_COMPLETION, -- 2.14.3 (Apple Git-98)