You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2012/08/25 01:40:13 UTC
svn commit: r1377151 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
Author: stack
Date: Fri Aug 24 23:40:13 2012
New Revision: 1377151
URL: http://svn.apache.org/viewvc?rev=1377151&view=rev
Log:
HBASE-6529 With HFile v2, the region server will always perform an extra copy of source files
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java?rev=1377151&r1=1377150&r2=1377151&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/Store.java Fri Aug 24 23:40:13 2012
@@ -52,6 +52,7 @@ import org.apache.hadoop.hbase.KeyValue;
import org.apache.hadoop.hbase.KeyValue.KVComparator;
import org.apache.hadoop.hbase.RemoteExceptionHandler;
import org.apache.hadoop.hbase.client.Scan;
+import org.apache.hadoop.hbase.fs.HFileSystem;
import org.apache.hadoop.hbase.io.HeapSize;
import org.apache.hadoop.hbase.io.hfile.CacheConfig;
import org.apache.hadoop.hbase.io.hfile.Compression;
@@ -534,7 +535,8 @@ public class Store extends SchemaConfigu
// Move the file if it's on another filesystem
FileSystem srcFs = srcPath.getFileSystem(conf);
- if (!srcFs.equals(fs)) {
+ FileSystem desFs = fs instanceof HFileSystem ? ((HFileSystem)fs).getBackingFs() : fs;
+ if (!srcFs.equals(desFs)) {
LOG.info("File " + srcPath + " on different filesystem than " +
"destination store - moving to this filesystem.");
Path tmpPath = getTmpPath();