Repository: hadoop Updated Branches: refs/heads/trunk 8455d7075 -> 39a9dc8e4
YARN-7052. RM SchedulingMonitor gives no indication why the spawned thread crashed. Contributed by Eric Payne Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/39a9dc8e Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/39a9dc8e Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/39a9dc8e Branch: refs/heads/trunk Commit: 39a9dc8e4a6e1d13658867ad756878d3dd6352b0 Parents: 8455d70 Author: Jason Lowe <[email protected]> Authored: Fri Aug 25 15:31:29 2017 -0500 Committer: Jason Lowe <[email protected]> Committed: Fri Aug 25 15:31: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/39a9dc8e/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]
