Repository: hadoop Updated Branches: refs/heads/branch-2 5a83ffa39 -> ea5bb5898
YARN-7052. RM SchedulingMonitor gives no indication why the spawned thread crashed. Contributed by Eric Payne (cherry picked from commit 39a9dc8e4a6e1d13658867ad756878d3dd6352b0) Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/ea5bb589 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/ea5bb589 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/ea5bb589 Branch: refs/heads/branch-2 Commit: ea5bb5898f4712dc18764bcf0e7859aee631f71d Parents: 5a83ffa Author: Jason Lowe <[email protected]> Authored: Fri Aug 25 15:31:29 2017 -0500 Committer: Jason Lowe <[email protected]> Committed: Fri Aug 25 15:32:29 2017 -0500 ---------------------------------------------------------------------- .../server/resourcemanager/monitor/SchedulingMonitor.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/ea5bb589/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/SchedulingMonitor.java ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/SchedulingMonitor.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/SchedulingMonitor.java index 1e3f691..631d1a0 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/SchedulingMonitor.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/monitor/SchedulingMonitor.java @@ -100,9 +100,11 @@ public class SchedulingMonitor extends AbstractService { try { //invoke the preemption policy invokePolicy(); - } catch (YarnRuntimeException e) { - LOG.error("YarnRuntimeException raised while executing preemption" - + " checker, skip this run..., exception=", e); + } catch (Throwable t) { + // The preemption monitor does not alter structures nor do structures + // persist across invocations. Therefore, log, skip, and retry. + LOG.error("Exception raised while executing preemption" + + " checker, skip this run..., exception=", t); } } } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
