YARN-5103. With NM recovery enabled, restarting NM multiple times results in AM restart. Contributed by Junping Du

(cherry picked from commit d1df0266cf)
This commit is contained in:
Jason Lowe 2016-05-23 15:15:27 +00:00
parent 867cd2f5a2
commit e8bd2a5be6
1 changed files with 4 additions and 3 deletions

View File

@ -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 class RecoveredContainerLaunch extends ContainerLaunch {
} 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);