You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by jl...@apache.org on 2017/08/25 20:32:13 UTC
hadoop git commit: YARN-7052. RM SchedulingMonitor gives no
indication why the spawned thread crashed. Contributed by Eric Payne
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 <jl...@apache.org>
Authored: Fri Aug 25 15:31:29 2017 -0500
Committer: Jason Lowe <jl...@apache.org>
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: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org