YARN-5103. With NM recovery enabled, restarting NM multiple times res…

…ults in AM restart. Contributed by Junping Du
lliiun-z · May 23, 2016 · d1df026 · d1df026
1 parent 6161d9b
commit d1df026
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/...he/hadoop/yarn/server/nodemanager/containermanager/launcher/RecoveredContainerLaunch.java b/...he/hadoop/yarn/server/nodemanager/containermanager/launcher/RecoveredContainerLaunch.java
@@ -19,6 +19,7 @@
 package org.apache.hadoop.yarn.server.nodemanager.containermanager.launcher;
 
 import java.io.File;
+import java.io.InterruptedIOException;
 import java.io.IOException;
 
 import org.apache.commons.logging.Log;
@@ -90,11 +91,11 @@ public Integer call() {
       } else {
         LOG.warn("Unable to locate pid file for container " + containerIdStr);
       }
-    } catch (IOException e) {
-        LOG.error("Unable to recover container " + containerIdStr, e);
-    } catch (InterruptedException e) {
+    } catch (InterruptedException | InterruptedIOException e) {
       LOG.warn("Interrupted while waiting for exit code from " + containerId);
       notInterrupted = false;
+    } catch (IOException e) {
+      LOG.error("Unable to recover container " + containerIdStr, e);
     } finally {
       if (notInterrupted) {
         this.completed.set(true);