You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ec...@apache.org on 2011/10/24 19:29:11 UTC
svn commit: r1188247 -
/incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java
Author: ecn
Date: Mon Oct 24 17:29:10 2011
New Revision: 1188247
URL: http://svn.apache.org/viewvc?rev=1188247&view=rev
Log:
ACCUMULO-65: check the return code for the final rename when bringing a minor compaction file online
Modified:
incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java
Modified: incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java
URL: http://svn.apache.org/viewvc/incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java?rev=1188247&r1=1188246&r2=1188247&view=diff
==============================================================================
--- incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java (original)
+++ incubator/accumulo/trunk/src/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java Mon Oct 24 17:29:10 2011
@@ -768,7 +768,9 @@ public class Tablet {
fs.delete(newDatafile, true);
}
- fs.rename(tmpDatafile, newDatafile);
+ if (!fs.rename(tmpDatafile, newDatafile)) {
+ throw new IOException("rename fails");
+ }
}
break;
} catch (IOException ioe) {