Repository: hadoop
Updated Branches:
  refs/heads/trunk 389bc6d3d -> 5aa7052e3


YARN-8054. Improve robustness of the LocalDirsHandlerService 
MonitoringTimerTask thread. Contributed by Jonathan Eagles


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/5aa7052e
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/5aa7052e
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/5aa7052e

Branch: refs/heads/trunk
Commit: 5aa7052e319c3273243dda9993fb6c2d776eb7cc
Parents: 389bc6d
Author: Jason Lowe <jl...@apache.org>
Authored: Wed Mar 21 17:19:20 2018 -0500
Committer: Jason Lowe <jl...@apache.org>
Committed: Wed Mar 21 17:19:20 2018 -0500

----------------------------------------------------------------------
 .../yarn/server/nodemanager/LocalDirsHandlerService.java      | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/5aa7052e/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LocalDirsHandlerService.java
----------------------------------------------------------------------
diff --git 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LocalDirsHandlerService.java
 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LocalDirsHandlerService.java
index c0630b2..621cabc 100644
--- 
a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LocalDirsHandlerService.java
+++ 
b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/LocalDirsHandlerService.java
@@ -166,7 +166,12 @@ public class LocalDirsHandlerService extends 
AbstractService {
 
     @Override
     public void run() {
-      checkDirs();
+      try {
+        checkDirs();
+      } catch (Throwable t) {
+        // Prevent uncaught exceptions from killing this thread
+        LOG.warn("Error while checking local directories: ", t);
+      }
     }
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-commits-h...@hadoop.apache.org

Reply via email to