[GitHub] [pulsar] codelipenghui commented on a change in pull request #12734: Add remote cluster info to replicator producer name.

[GitBox <gi...@apache.org>]

codelipenghui commented on a change in pull request #12734:
URL: https://github.com/apache/pulsar/pull/12734#discussion_r747604005



##########
File path: pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractReplicator.java
##########
@@ -83,7 +83,8 @@ public AbstractReplicator(String topicName, String replicatorPrefix, String loca
                 .enableBatching(false)
                 .sendTimeout(0, TimeUnit.SECONDS) //
                 .maxPendingMessages(producerQueueSize) //
-                .producerName(getReplicatorName(replicatorPrefix, localCluster));
+                .producerName(String.format("%s%s%s", getReplicatorName(replicatorPrefix, localCluster),
+                        REPL_PRODUCER_NAME_DELIMITER, remoteCluster));

Review comment:
       This will change the producer that used to connect to the remote cluster right?




-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org