hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mberto...@apache.org
Subject [1/3] hbase git commit: HBASE-13578 Remove Arrays.asList().subList() from FSHLog.offer()
Date Tue, 28 Apr 2015 20:47:34 GMT
Repository: hbase
Updated Branches:
  refs/heads/branch-1 d9bf56ab3 -> 3a9304e98
  refs/heads/branch-1.1 da9f06318 -> 3fba9de03
  refs/heads/master d2c19416a -> ee562faeb


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/ee562fae
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ee562fae
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ee562fae

Branch: refs/heads/master
Commit: ee562faeb3cd0b7a49a1319fdc775f1d43e32341
Parents: d2c1941
Author: Matteo Bertozzi <matteo.bertozzi@cloudera.com>
Authored: Tue Apr 28 20:43:08 2015 +0100
Committer: Matteo Bertozzi <matteo.bertozzi@cloudera.com>
Committed: Tue Apr 28 20:43:08 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/ee562fae/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 9408f99..549f0ce 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
@@ -1248,7 +1248,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]);
+      }
     }
 
     /**


Mime
View raw message