[GitHub] [pulsar-site] xiaogang-sn commented on a diff in pull request #510: [improve][doc] Add instructions for enabling replicated subscription

["xiaogang-sn (via GitHub)" <gi...@apache.org>]

xiaogang-sn commented on code in PR #510:
URL: https://github.com/apache/pulsar-site/pull/510#discussion_r1161590735


##########
docs/client-libraries-cluster-level-failover.md:
##########
@@ -26,7 +26,8 @@ For more information about cluster-level failover, including concepts, benefits,
   * The number of BookKeeper nodes should be equal to or greater than the ensemble quorum.
   * The number of ZooKeeper nodes should be equal to or greater than 3.
 * **Turn on geo-replication** between the primary cluster and any dependent cluster (primary to backup or backup to backup) to prevent data loss.
-* Set `replicateSubscriptionState` to `true` when creating consumers.
+* Set `enableReplicatedSubscriptions` to `true` in [`broker.conf`](https://github.com/apache/pulsar/blob/470b674016c8718f2dfd0a0f93cf02d49af0fead/conf/broker.conf#L592).

Review Comment:
   Can we simply say "enable replicated subscription" here with the link pointing to https://pulsar.apache.org/docs/2.11.x/administration-geo/#enable-replicated-subscription which will include both broker configuration and consumer configuration?



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