Jennifer88huang commented on a change in pull request #11466:
URL: https://github.com/apache/pulsar/pull/11466#discussion_r677934061
##########
File path: site2/docs/deploy-bare-metal-multi-cluster.md
##########
@@ -127,15 +124,15 @@ $ bin/pulsar-daemon start zookeeper
### Deploy the configuration store
-The ZooKeeper cluster that is configured and started up in the section above
is a *local* ZooKeeper cluster that you can use to manage a single Pulsar
cluster. In addition to a local cluster, however, a full Pulsar instance also
requires a configuration store for handling some instance-level configuration
and coordination tasks.
+The ZooKeeper cluster that is configured and started up in the section above
is a local ZooKeeper cluster that you can use to manage a single Pulsar
cluster. In addition to a local cluster, however, a full Pulsar instance also
requires a configuration store for handling some instance-level configuration
and coordination tasks.
-If you deploy a [single-cluster](#single-cluster-pulsar-instance) instance,
you do not need a separate cluster for the configuration store. If, however,
you deploy a [multi-cluster](#multi-cluster-pulsar-instance) instance, you
should stand up a separate ZooKeeper cluster for configuration tasks.
+If you deploy a single-cluster instance, you do not need a separate cluster
for the configuration store. If, however, you deploy a multi-cluster instance,
you should stand up a separate ZooKeeper cluster for configuration tasks.
#### Single-cluster Pulsar instance
If your Pulsar instance consists of just one cluster, then you can deploy a
configuration store on the same machines as the local ZooKeeper quorum but run
on different TCP ports.
-To deploy a ZooKeeper configuration store in a single-cluster instance, add
the same ZooKeeper servers that the local quorom uses to the configuration file
in
[`conf/global_zookeeper.conf`](reference-configuration.md#configuration-store)
using the same method for [local ZooKeeper](#local-zookeeper), but make sure to
use a different port (2181 is the default for ZooKeeper). The following is an
example that uses port 2184 for a three-node ZooKeeper cluster:
+To deploy a ZooKeeper configuration store in a single-cluster instance, add
the same ZooKeeper servers that the local quorom uses to the configuration file
in
[`conf/global_zookeeper.conf`](reference-configuration.md#configuration-store)
using the same method for local ZooKeeper](#local-zookeeper), but make sure to
use a different port (2181 is the default for ZooKeeper). The following is an
example that uses port 2184 for a three-node ZooKeeper cluster:
Review comment:
1. The first sentence is too long, try to use shorter sentences, less
than 25 words for a full sentence.
2. the link for "local ZooKeeper](#local-zookeeper)" is not correct, try to
fix it.
--
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-unsubscr...@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
