hadoop-common-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jl...@apache.org
Subject hadoop git commit: YARN-7052. RM SchedulingMonitor gives no indication why the spawned thread crashed. Contributed by Eric Payne
Date Fri, 25 Aug 2017 20:32:36 GMT
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 <jlowe@apache.org>
Authored: Fri Aug 25 15:31:29 2017 -0500
Committer: Jason Lowe <jlowe@apache.org>
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: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org


Mime
View raw message