svn commit: r1526393 - in /hadoop/common/branches/branch-1: CHANGES.txt src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java

[jing9]

Author: jing9
Date: Thu Sep 26 06:10:23 2013
New Revision: 1526393

URL: http://svn.apache.org/r1526393
Log:
HDFS-5245. shouldRetry() in WebHDFSFileSystem generates excessive warnings. 
Contributed by Haohui Mai.

Modified:
    hadoop/common/branches/branch-1/CHANGES.txt
    
hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java

Modified: hadoop/common/branches/branch-1/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-1/CHANGES.txt?rev=1526393&r1=1526392&r2=1526393&view=diff
==============================================================================
--- hadoop/common/branches/branch-1/CHANGES.txt (original)
+++ hadoop/common/branches/branch-1/CHANGES.txt Thu Sep 26 06:10:23 2013
@@ -40,6 +40,9 @@ Release 1.3.0 - unreleased
     HDFS-5038. Backport several branch-2 APIs (FileSystem#newInstance, 
     DFSClient#getNamenode, and FileStatus#isDirectory) to branch-1. (jing9)
 
+    HDFS-5245. shouldRetry() in WebHDFSFileSystem generates excessive warnings.
+    (Haohui Mai via jing9)
+
   BUG FIXES
 
     HADOOP-9863. Backport HADOOP-8686 to support BigEndian on ppc64. 

Modified: 
hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java?rev=1526393&r1=1526392&r2=1526393&view=diff
==============================================================================
--- 
hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
 (original)
+++ 
hadoop/common/branches/branch-1/src/hdfs/org/apache/hadoop/hdfs/web/WebHdfsFileSystem.java
 Thu Sep 26 06:10:23 2013
@@ -502,7 +502,11 @@ public class WebHdfsFileSystem extends F
             return;
           }
         } catch(Exception e) {
-          LOG.warn("Original exception is ", ioe);
+          /**
+           * In this case the retry policy has decided that it won't retry.
+           * Swallow the exception to avoid excessive warnings in the logs.
+           */
+          // LOG.warn("Original exception is ", ioe);
           throw toIOException(e);
         }
       }