You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by mb...@apache.org on 2015/04/28 22:47:35 UTC

[2/3] hbase git commit: HBASE-13578 Remove Arrays.asList().subList() from FSHLog.offer()

HBASE-13578 Remove Arrays.asList().subList() from FSHLog.offer()


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/3a9304e9
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3a9304e9
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3a9304e9

Branch: refs/heads/branch-1
Commit: 3a9304e98f7e5a72649ad2aa45574a6f6e53db4d
Parents: d9bf56a
Author: Matteo Bertozzi <ma...@cloudera.com>
Authored: Tue Apr 28 20:43:08 2015 +0100
Committer: Matteo Bertozzi <ma...@cloudera.com>
Committed: Tue Apr 28 21:13:40 2015 +0100

----------------------------------------------------------------------
 .../java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/3a9304e9/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java
index 91876b1..aa722a0 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/FSHLog.java
@@ -1249,7 +1249,9 @@ public class FSHLog implements WAL {
     void offer(final long sequence, final SyncFuture [] syncFutures, final int syncFutureCount) {
       // Set sequence first because the add to the queue will wake the thread if sleeping.
       this.sequence = sequence;
-      this.syncFutures.addAll(Arrays.asList(syncFutures).subList(0, syncFutureCount));
+      for (int i = 0; i < syncFutureCount; ++i) {
+        this.syncFutures.add(syncFutures[i]);
+      }
     }
 
     /**