You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-commits@hadoop.apache.org by sz...@apache.org on 2009/10/23 02:28:16 UTC

svn commit: r828900 - in /hadoop/hdfs/trunk: CHANGES.txt src/java/org/apache/hadoop/hdfs/DFSClient.java

Author: szetszwo
Date: Fri Oct 23 00:28:15 2009
New Revision: 828900

URL: http://svn.apache.org/viewvc?rev=828900&view=rev
Log:
HDFS-127. Reset failure count in DFSClient for each block acquiring operation.  Contributed by Igor Bolotin

Modified:
    hadoop/hdfs/trunk/CHANGES.txt
    hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/DFSClient.java

Modified: hadoop/hdfs/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/CHANGES.txt?rev=828900&r1=828899&r2=828900&view=diff
==============================================================================
--- hadoop/hdfs/trunk/CHANGES.txt (original)
+++ hadoop/hdfs/trunk/CHANGES.txt Fri Oct 23 00:28:15 2009
@@ -439,6 +439,11 @@
 
 Release 0.20.2 - Unreleased
 
+  IMPROVEMENTS
+
+    HDFS-127. Reset failure count in DFSClient for each block acquiring
+    operation.  (Igor Bolotin via szetszwo)
+
   BUG FIXES
 
     HDFS-686. NullPointerException is thrown while merging edit log and image.

Modified: hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/DFSClient.java
URL: http://svn.apache.org/viewvc/hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/DFSClient.java?rev=828900&r1=828899&r2=828900&view=diff
==============================================================================
--- hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/DFSClient.java (original)
+++ hadoop/hdfs/trunk/src/java/org/apache/hadoop/hdfs/DFSClient.java Fri Oct 23 00:28:15 2009
@@ -1879,6 +1879,8 @@
       //
       DatanodeInfo chosenNode = null;
       int refetchToken = 1; // only need to get a new access token once
+      failures = 0;
+      
       while (true) {
         //
         // Compute desired block
@@ -2108,6 +2110,7 @@
       //
       Socket dn = null;
       int refetchToken = 1; // only need to get a new access token once
+      failures = 0;
       
       while (true) {
         // cached block locations may have been updated by chooseDataNode()