Repository: hadoop Updated Branches: refs/heads/branch-2.8.2 85e5fd94b -> bc91e2310
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/bc91e231 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/bc91e231 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/bc91e231 Branch: refs/heads/branch-2.8.2 Commit: bc91e2310f17d8e1688db00dc0550e1a79a17ca1 Parents: 85e5fd9 Author: Jason Lowe <[email protected]> Authored: Fri Aug 25 15:31:29 2017 -0500 Committer: Jason Lowe <[email protected]> Committed: Fri Aug 25 15:35:04 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/bc91e231/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 3def27f..4cc8989 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 @@ -102,9 +102,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]
