HBASE-19905 ReplicationSyncUp tool will not exit if a peer replication is disabled
Signed-off-by: Ashish Singhi <ashishsinghi@apache.org>
This commit is contained in:
parent
b0e998f2a5
commit
397d34736e
|
@ -55,6 +55,7 @@ import org.apache.hadoop.hbase.client.ConnectionFactory;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationException;
|
import org.apache.hadoop.hbase.replication.ReplicationException;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationListener;
|
import org.apache.hadoop.hbase.replication.ReplicationListener;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationPeer;
|
import org.apache.hadoop.hbase.replication.ReplicationPeer;
|
||||||
|
import org.apache.hadoop.hbase.replication.ReplicationPeer.PeerState;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationPeers;
|
import org.apache.hadoop.hbase.replication.ReplicationPeers;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationQueueInfo;
|
import org.apache.hadoop.hbase.replication.ReplicationQueueInfo;
|
||||||
import org.apache.hadoop.hbase.replication.ReplicationQueueStorage;
|
import org.apache.hadoop.hbase.replication.ReplicationQueueStorage;
|
||||||
|
@ -747,6 +748,13 @@ public class ReplicationSourceManager implements ReplicationListener {
|
||||||
abortWhenFail(() -> queueStorage.removeQueue(server.getServerName(), queueId));
|
abortWhenFail(() -> queueStorage.removeQueue(server.getServerName(), queueId));
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
if (server instanceof ReplicationSyncUp.DummyServer
|
||||||
|
&& peer.getPeerState().equals(PeerState.DISABLED)) {
|
||||||
|
LOG.warn("Peer {} is disbaled. ReplicationSyncUp tool will skip "
|
||||||
|
+ "replicating data to this peer.",
|
||||||
|
actualPeerId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
// track sources in walsByIdRecoveredQueues
|
// track sources in walsByIdRecoveredQueues
|
||||||
Map<String, SortedSet<String>> walsByGroup = new HashMap<>();
|
Map<String, SortedSet<String>> walsByGroup = new HashMap<>();
|
||||||
walsByIdRecoveredQueues.put(queueId, walsByGroup);
|
walsByIdRecoveredQueues.put(queueId, walsByGroup);
|
||||||
|
|
Loading…
Reference in New Issue