Author: dhruba
Date: Thu Jun 17 01:52:50 2010
New Revision: 955448
URL: http://svn.apache.org/viewvc?rev=955448&view=rev
Log:
HDFS-1210. DFSClient should log exception when block recovery fails.
(Todd Lipcon via dhruba)
Modified:
hadoop/common/branches/branch-0.20-append/CHANGES.txt
hadoop/common/branches/branch-0.20-append/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
Modified: hadoop/common/branches/branch-0.20-append/CHANGES.txt
URL:
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-append/CHANGES.txt?rev=955448&r1=955447&r2=955448&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-append/CHANGES.txt (original)
+++ hadoop/common/branches/branch-0.20-append/CHANGES.txt Thu Jun 17 01:52:50
2010
@@ -59,6 +59,9 @@ Release 0.20-append - Unreleased
HDFS-1204. Lease expiration should recover single files,
not entire lease holder (Sam Rash via dhruba)
+ HDFS-1210. DFSClient should log exception when block recovery fails.
+ (Todd Lipcon via dhruba)
+
Release 0.20.3 - Unreleased
NEW FEATURES
Modified:
hadoop/common/branches/branch-0.20-append/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
URL:
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-append/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java?rev=955448&r1=955447&r2=955448&view=diff
==============================================================================
---
hadoop/common/branches/branch-0.20-append/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
(original)
+++
hadoop/common/branches/branch-0.20-append/src/hdfs/org/apache/hadoop/hdfs/DFSClient.java
Thu Jun 17 01:52:50 2010
@@ -2635,6 +2635,8 @@ public class DFSClient implements FSCons
primary = createClientDatanodeProtocolProxy(primaryNode, conf);
newBlock = primary.recoverBlock(block, isAppend, newnodes);
} catch (IOException e) {
+ LOG.warn("Failed recovery attempt #" + recoveryErrorCount +
+ " from primary datanode " + primaryNode, e);
recoveryErrorCount++;
if (recoveryErrorCount > maxRecoveryErrorCount) {
if (nodes.length > 1) {