HBASE-22601 Misconfigured addition of peers leads to cluster shutdown. (#500)

shahrs87 · Apache9 · commit 0a1950f43cbf · 2019-08-21T21:56:04.000+08:00
Signed-off-by: Duo Zhang &lt;zhangduo@apache.org&gt;
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSource.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSource.java
@@ -510,6 +510,10 @@ private void initialize() {
       }
     }
 
+    if (!this.isSourceActive()) {
+      return;
+    }
+
     // In rare case, zookeeper setting may be messed up. That leads to the incorrect
     // peerClusterId value, which is the same as the source clusterId
     if (clusterId.equals(peerClusterId) && !replicationEndpoint.canReplicateToSameCluster()) {

Original file line number	Diff line number	Diff line change
`@@ -510,6 +510,10 @@ private void initialize() {`
`510`	`510`	`}`
`511`	`511`	`}`
`512`	`512`
	`513`	`+ if (!this.isSourceActive()) {`
	`514`	`+ return;`
	`515`	`+ }`
	`516`	`+`
`513`	`517`	`// In rare case, zookeeper setting may be messed up. That leads to the incorrect`
`514`	`518`	`// peerClusterId value, which is the same as the source clusterId`
`515`	`519`	`if (clusterId.equals(peerClusterId) && !replicationEndpoint.canReplicateToSameCluster()) {`