You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2014/03/25 00:37:16 UTC

[3/5] git commit: ACCUMULO-2440 Use the correctBufferSize method to (correctly) fix the bufferSize when not provided.

ACCUMULO-2440 Use the correctBufferSize method to (correctly) fix the bufferSize when not provided.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/62ce7524
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/62ce7524
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/62ce7524

Branch: refs/heads/master
Commit: 62ce7524f4618e1b46166a6d8f4f65fbbda4c4ec
Parents: bc1f026
Author: Josh Elser <el...@apache.org>
Authored: Mon Mar 24 16:32:10 2014 -0700
Committer: Josh Elser <el...@apache.org>
Committed: Mon Mar 24 16:32:10 2014 -0700

----------------------------------------------------------------------
 .../org/apache/accumulo/server/fs/VolumeManagerImpl.java     | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/62ce7524/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
index 6a3140b..8fe6579 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
@@ -151,11 +151,9 @@ public class VolumeManagerImpl implements VolumeManager {
 
     Volume v = getVolumeByPath(path);
     FileSystem fs = v.getFileSystem();
-
-    if (bufferSize == 0) {
-      fs.getConf().getInt("io.file.buffer.size", 4096);
-    }
-    return fs.create(path, overwrite, bufferSize, replication, correctBlockSize(fs.getConf(), blockSize));
+    blockSize = correctBlockSize(fs.getConf(), blockSize);
+    bufferSize = correctBufferSize(fs.getConf(), bufferSize);
+    return fs.create(path, overwrite, bufferSize, replication, blockSize);
   }
 
   @Override