Repository: hadoop
Updated Branches:
  refs/heads/HDFS-12996 e37cde0a6 -> 6c9af6c88


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/6c9af6c8
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/6c9af6c8
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/6c9af6c8

Branch: refs/heads/HDFS-12996
Commit: 6c9af6c88f337a3573a01fc065adcf09a9d01b2b
Parents: fcec503
Author: Jason Lowe <jl...@apache.org>
Authored: Wed Mar 21 17:19:20 2018 -0500
Committer: Hanisha Koneru <hanishakon...@apache.org>
Committed: Wed Mar 21 16:46:52 2018 -0700

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


http://git-wip-us.apache.org/repos/asf/hadoop/blob/6c9af6c8/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