[16/28] hbase git commit: HBASE-19926 Use a separated class to implement the WALActionListener for Replication
HBASE-19926 Use a separated class to implement the WALActionListener for Replication Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/14420e1b Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/14420e1b Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/14420e1b Branch: refs/heads/HBASE-19064 Commit: 14420e1b415cd468f652bf0137bda575e0a5980a Parents: 397d347 Author: zhangduo Authored: Sun Feb 4 10:42:33 2018 +0800 Committer: zhangduo Committed: Sun Feb 4 20:36:43 2018 +0800 -- .../replication/regionserver/Replication.java | 22 + .../regionserver/ReplicationSourceManager.java | 45 - .../ReplicationSourceWALActionListener.java | 98 .../TestReplicationSourceManager.java | 27 +- 4 files changed, 105 insertions(+), 87 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hbase/blob/14420e1b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java index aaf3beb..7803ac4 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java @@ -34,7 +34,6 @@ import org.apache.hadoop.hbase.Server; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.regionserver.ReplicationSinkService; import org.apache.hadoop.hbase.regionserver.ReplicationSourceService; -import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener; import org.apache.hadoop.hbase.replication.ReplicationFactory; import org.apache.hadoop.hbase.replication.ReplicationPeers; import org.apache.hadoop.hbase.replication.ReplicationQueueStorage; @@ -42,8 +41,6 @@ import org.apache.hadoop.hbase.replication.ReplicationStorageFactory; import org.apache.hadoop.hbase.replication.ReplicationTracker; import org.apache.hadoop.hbase.replication.ReplicationUtils; import org.apache.hadoop.hbase.util.Pair; -import org.apache.hadoop.hbase.wal.WALEdit; -import org.apache.hadoop.hbase.wal.WALKey; import org.apache.hadoop.hbase.wal.WALProvider; import org.apache.hadoop.hbase.zookeeper.ZKClusterId; import org.apache.yetus.audience.InterfaceAudience; @@ -127,23 +124,8 @@ public class Replication implements ReplicationSourceService, ReplicationSinkSer replicationTracker, conf, this.server, fs, logDir, oldLogDir, clusterId, walProvider != null ? walProvider.getWALFileLengthProvider() : p -> OptionalLong.empty()); if (walProvider != null) { - walProvider.addWALActionsListener(new WALActionsListener() { - -@Override -public void preLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.preLogRoll(newPath); -} - -@Override -public void postLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.postLogRoll(newPath); -} - -@Override -public void visitLogEntryBeforeWrite(WALKey logKey, WALEdit logEdit) throws IOException { - replicationManager.scopeWALEdits(logKey, logEdit); -} - }); + walProvider +.addWALActionsListener(new ReplicationSourceWALActionListener(conf, replicationManager)); } this.statsThreadPeriod = this.conf.getInt("replication.stats.thread.period.seconds", 5 * 60); http://git-wip-us.apache.org/repos/asf/hbase/blob/14420e1b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java index 6e87563..85b2e85 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java @@ -43,8 +43,6 @@ import java.util.stream.Collectors; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; -import org.apache.hadoop.hbase.Cell; -import org.apache.hadoop.hbase.CellUtil; import org.apache.hadoop.hbase.HConstants; import org.apache.hadoop.hbase.MetaTableAccessor; import org.apache.hadoop.hbase.Server; @@ -60,13 +58,9 @@ import org.apache.hado
[09/46] hbase git commit: HBASE-19926 Use a separated class to implement the WALActionListener for Replication
HBASE-19926 Use a separated class to implement the WALActionListener for Replication Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/3b603d2c Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3b603d2c Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3b603d2c Branch: refs/heads/HBASE-19397-branch-2 Commit: 3b603d2c08c1f1905a589597737412b43970a304 Parents: 0ca7a2e Author: zhangduo Authored: Sun Feb 4 10:42:33 2018 +0800 Committer: zhangduo Committed: Sun Feb 4 20:32:14 2018 +0800 -- .../replication/regionserver/Replication.java | 22 + .../regionserver/ReplicationSourceManager.java | 47 +- .../ReplicationSourceWALActionListener.java | 98 .../TestReplicationSourceManager.java | 30 ++ 4 files changed, 108 insertions(+), 89 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hbase/blob/3b603d2c/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java index 0274b0a..ad12c66 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java @@ -35,7 +35,6 @@ import org.apache.hadoop.hbase.Server; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.regionserver.ReplicationSinkService; import org.apache.hadoop.hbase.regionserver.ReplicationSourceService; -import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener; import org.apache.hadoop.hbase.replication.ReplicationException; import org.apache.hadoop.hbase.replication.ReplicationFactory; import org.apache.hadoop.hbase.replication.ReplicationPeers; @@ -44,8 +43,6 @@ import org.apache.hadoop.hbase.replication.ReplicationQueuesArguments; import org.apache.hadoop.hbase.replication.ReplicationTracker; import org.apache.hadoop.hbase.replication.ReplicationUtils; import org.apache.hadoop.hbase.util.Pair; -import org.apache.hadoop.hbase.wal.WALEdit; -import org.apache.hadoop.hbase.wal.WALKey; import org.apache.hadoop.hbase.wal.WALProvider; import org.apache.hadoop.hbase.zookeeper.ZKClusterId; import org.apache.yetus.audience.InterfaceAudience; @@ -130,23 +127,8 @@ public class Replication implements ReplicationSourceService, ReplicationSinkSer replicationTracker, conf, this.server, fs, logDir, oldLogDir, clusterId, walProvider != null ? walProvider.getWALFileLengthProvider() : p -> OptionalLong.empty()); if (walProvider != null) { - walProvider.addWALActionsListener(new WALActionsListener() { - -@Override -public void preLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.preLogRoll(newPath); -} - -@Override -public void postLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.postLogRoll(newPath); -} - -@Override -public void visitLogEntryBeforeWrite(WALKey logKey, WALEdit logEdit) throws IOException { - replicationManager.scopeWALEdits(logKey, logEdit); -} - }); + walProvider +.addWALActionsListener(new ReplicationSourceWALActionListener(conf, replicationManager)); } this.statsThreadPeriod = this.conf.getInt("replication.stats.thread.period.seconds", 5 * 60); http://git-wip-us.apache.org/repos/asf/hbase/blob/3b603d2c/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java index 8543896..cbbfca0 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java @@ -40,12 +40,9 @@ import java.util.concurrent.ThreadLocalRandom; import java.util.concurrent.ThreadPoolExecutor; import java.util.concurrent.TimeUnit; import java.util.concurrent.atomic.AtomicLong; - import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; -import org.apache.hadoop.hbase.Cell; -import org.apache.hadoop.hbase.CellUtil; import org.apache
hbase git commit: HBASE-19926 Use a separated class to implement the WALActionListener for Replication
Repository: hbase Updated Branches: refs/heads/branch-2 0ca7a2e91 -> 3b603d2c0 HBASE-19926 Use a separated class to implement the WALActionListener for Replication Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/3b603d2c Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3b603d2c Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3b603d2c Branch: refs/heads/branch-2 Commit: 3b603d2c08c1f1905a589597737412b43970a304 Parents: 0ca7a2e Author: zhangduo Authored: Sun Feb 4 10:42:33 2018 +0800 Committer: zhangduo Committed: Sun Feb 4 20:32:14 2018 +0800 -- .../replication/regionserver/Replication.java | 22 + .../regionserver/ReplicationSourceManager.java | 47 +- .../ReplicationSourceWALActionListener.java | 98 .../TestReplicationSourceManager.java | 30 ++ 4 files changed, 108 insertions(+), 89 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hbase/blob/3b603d2c/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java index 0274b0a..ad12c66 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java @@ -35,7 +35,6 @@ import org.apache.hadoop.hbase.Server; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.regionserver.ReplicationSinkService; import org.apache.hadoop.hbase.regionserver.ReplicationSourceService; -import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener; import org.apache.hadoop.hbase.replication.ReplicationException; import org.apache.hadoop.hbase.replication.ReplicationFactory; import org.apache.hadoop.hbase.replication.ReplicationPeers; @@ -44,8 +43,6 @@ import org.apache.hadoop.hbase.replication.ReplicationQueuesArguments; import org.apache.hadoop.hbase.replication.ReplicationTracker; import org.apache.hadoop.hbase.replication.ReplicationUtils; import org.apache.hadoop.hbase.util.Pair; -import org.apache.hadoop.hbase.wal.WALEdit; -import org.apache.hadoop.hbase.wal.WALKey; import org.apache.hadoop.hbase.wal.WALProvider; import org.apache.hadoop.hbase.zookeeper.ZKClusterId; import org.apache.yetus.audience.InterfaceAudience; @@ -130,23 +127,8 @@ public class Replication implements ReplicationSourceService, ReplicationSinkSer replicationTracker, conf, this.server, fs, logDir, oldLogDir, clusterId, walProvider != null ? walProvider.getWALFileLengthProvider() : p -> OptionalLong.empty()); if (walProvider != null) { - walProvider.addWALActionsListener(new WALActionsListener() { - -@Override -public void preLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.preLogRoll(newPath); -} - -@Override -public void postLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.postLogRoll(newPath); -} - -@Override -public void visitLogEntryBeforeWrite(WALKey logKey, WALEdit logEdit) throws IOException { - replicationManager.scopeWALEdits(logKey, logEdit); -} - }); + walProvider +.addWALActionsListener(new ReplicationSourceWALActionListener(conf, replicationManager)); } this.statsThreadPeriod = this.conf.getInt("replication.stats.thread.period.seconds", 5 * 60); http://git-wip-us.apache.org/repos/asf/hbase/blob/3b603d2c/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java index 8543896..cbbfca0 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java @@ -40,12 +40,9 @@ import java.util.concurrent.ThreadLocalRandom; import java.util.concurrent.ThreadPoolExecutor; import java.util.concurrent.TimeUnit; import java.util.concurrent.atomic.AtomicLong; - import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; -import org.apache.hadoop.h
hbase git commit: HBASE-19926 Use a separated class to implement the WALActionListener for Replication
Repository: hbase Updated Branches: refs/heads/master 397d34736 -> 14420e1b4 HBASE-19926 Use a separated class to implement the WALActionListener for Replication Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/14420e1b Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/14420e1b Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/14420e1b Branch: refs/heads/master Commit: 14420e1b415cd468f652bf0137bda575e0a5980a Parents: 397d347 Author: zhangduo Authored: Sun Feb 4 10:42:33 2018 +0800 Committer: zhangduo Committed: Sun Feb 4 20:36:43 2018 +0800 -- .../replication/regionserver/Replication.java | 22 + .../regionserver/ReplicationSourceManager.java | 45 - .../ReplicationSourceWALActionListener.java | 98 .../TestReplicationSourceManager.java | 27 +- 4 files changed, 105 insertions(+), 87 deletions(-) -- http://git-wip-us.apache.org/repos/asf/hbase/blob/14420e1b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java index aaf3beb..7803ac4 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/Replication.java @@ -34,7 +34,6 @@ import org.apache.hadoop.hbase.Server; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.regionserver.ReplicationSinkService; import org.apache.hadoop.hbase.regionserver.ReplicationSourceService; -import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener; import org.apache.hadoop.hbase.replication.ReplicationFactory; import org.apache.hadoop.hbase.replication.ReplicationPeers; import org.apache.hadoop.hbase.replication.ReplicationQueueStorage; @@ -42,8 +41,6 @@ import org.apache.hadoop.hbase.replication.ReplicationStorageFactory; import org.apache.hadoop.hbase.replication.ReplicationTracker; import org.apache.hadoop.hbase.replication.ReplicationUtils; import org.apache.hadoop.hbase.util.Pair; -import org.apache.hadoop.hbase.wal.WALEdit; -import org.apache.hadoop.hbase.wal.WALKey; import org.apache.hadoop.hbase.wal.WALProvider; import org.apache.hadoop.hbase.zookeeper.ZKClusterId; import org.apache.yetus.audience.InterfaceAudience; @@ -127,23 +124,8 @@ public class Replication implements ReplicationSourceService, ReplicationSinkSer replicationTracker, conf, this.server, fs, logDir, oldLogDir, clusterId, walProvider != null ? walProvider.getWALFileLengthProvider() : p -> OptionalLong.empty()); if (walProvider != null) { - walProvider.addWALActionsListener(new WALActionsListener() { - -@Override -public void preLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.preLogRoll(newPath); -} - -@Override -public void postLogRoll(Path oldPath, Path newPath) throws IOException { - replicationManager.postLogRoll(newPath); -} - -@Override -public void visitLogEntryBeforeWrite(WALKey logKey, WALEdit logEdit) throws IOException { - replicationManager.scopeWALEdits(logKey, logEdit); -} - }); + walProvider +.addWALActionsListener(new ReplicationSourceWALActionListener(conf, replicationManager)); } this.statsThreadPeriod = this.conf.getInt("replication.stats.thread.period.seconds", 5 * 60); http://git-wip-us.apache.org/repos/asf/hbase/blob/14420e1b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java -- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java index 6e87563..85b2e85 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSourceManager.java @@ -43,8 +43,6 @@ import java.util.stream.Collectors; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; -import org.apache.hadoop.hbase.Cell; -import org.apache.hadoop.hbase.CellUtil; import org.apache.hadoop.hbase.HConstants; import org.apache.hadoop.hbase.MetaTableAccessor; impo