infraio commented on a change in pull request #2064:
URL: https://github.com/apache/hbase/pull/2064#discussion_r455448274
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSource.java
##########
@@ -139,17 +144,16 @@
* Instantiation method used by region servers
* @param conf configuration to use
* @param fs file system to use
- * @param manager replication manager to ping to
* @param server the server for this region server
* @param queueId the id of our replication queue
* @param clusterId unique UUID for the cluster
* @param metrics metrics for replication source
*/
@Override
- public void init(Configuration conf, FileSystem fs, ReplicationSourceManager
manager,
+ public void init(Configuration conf, FileSystem fs, Path walDir,
ReplicationSourceManager manager,
Review comment:
There are one typre ReplicationSource which called
RecoveredReplicationSource, which replicate the WAL queues from a dead
RegionServer. For RecoveredReplicationSource, the replication stream will
finished when all WALs was replicated because the dead server cannot generate
new WAL. Here need manager because RecoveredReplicationSource need to finish
itself and clear the related reference from ReplicationSourceManager. When move
to ReplicationServer future, need a ReplicationServer instead of this manager.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
