You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2013/02/27 20:09:58 UTC
svn commit: r1450924 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java
Author: tedyu
Date: Wed Feb 27 19:09:57 2013
New Revision: 1450924
URL: http://svn.apache.org/r1450924
Log:
HBASE-7878 Addendum for trying fs.append in case recoverLease throws exception
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java?rev=1450924&r1=1450923&r2=1450924&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/util/FSHDFSUtils.java Wed Feb 27 19:09:57 2013
@@ -94,12 +94,9 @@ public class FSHDFSUtils extends FSUtils
// function was properly called, but threw it's own exception
throw (IOException) ite.getCause();
} catch (Exception e) {
- // hdfs 2.0 may throw RecoveryInProgressException
- if (!e.getClass().getName().contains("RecoveryInProgressException")) {
- LOG.debug("Failed fs.recoverLease invocation, " + e.toString() +
- ", trying fs.append instead");
- ex = e;
- }
+ LOG.debug("Failed fs.recoverLease invocation, " + e.toString() +
+ ", trying fs.append instead");
+ ex = e;
}
if (ex != null || System.currentTimeMillis() - startWaiting > recoveryTimeout) {
ex = null; // assume the following append() call would succeed