Jennifer88huang commented on a change in pull request #11452: URL: https://github.com/apache/pulsar/pull/11452#discussion_r676240421
########## File path: site2/docs/deploy-bare-metal.md ########## @@ -65,7 +65,7 @@ When you deploy a Pulsar cluster, keep in mind the following basic better choice #### ZooKeeper -For machines running ZooKeeper, is is recommended to use less powerful machines or VMs. Pulsar uses ZooKeeper only for periodic coordination-related and configuration-related tasks, *not* for basic operations. If you run Pulsar on [Amazon Web Services](https://aws.amazon.com/) (AWS), for example, a [t2.small](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/t2-instances.html) instance might likely suffice. +For machines running ZooKeeper, it is recommended to use less powerful machines or VMs. Pulsar uses ZooKeeper only for periodic coordination-related and configuration-related tasks, not for basic operations. If you run Pulsar on [Amazon Web Services](https://aws.amazon.com/) (AWS), for example, a [t2.small](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/t2-instances.html) instance might likely suffice. Review comment: For machines running ZooKeeper, it is recommended to use less powerful machines or VMs. Pulsar uses ZooKeeper only for periodic coordination-related and configuration-related tasks, not for basic operations. If you run Pulsar on [Amazon Web Services (AWS)](https://aws.amazon.com/), for example, a [t2.small](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/t2-instances.html)... ########## File path: site2/docs/deploy-bare-metal.md ########## @@ -30,10 +30,10 @@ Deploying a Pulsar cluster involves doing the following (in order): ### Requirements -Currently, Pulsar is available for 64-bit **macOS*, **Linux**, and **Windows**. To use Pulsar, you need to install 64-bit JRE/JDK 8 or later versions. +Currently, Pulsar is available for 64-bit **macOS**, **Linux**, and **Windows**. To use Pulsar, you need to install 64-bit JRE/JDK 8 or later versions. -> If you already have an existing ZooKeeper cluster and want to reuse it, you do not need to prepare the machines -> for running ZooKeeper. +> If you already have an existing ZooKeeper cluster and want to reuse it, you do not need to prepare additional machines +> running ZooKeeper. Review comment: ```suggestion > If you want to reuse an existing ZooKeeper cluster, you do not need to prepare additional machines running ZooKeeper. ``` ########## File path: site2/docs/deploy-bare-metal.md ########## @@ -76,14 +76,14 @@ For machines running a bookie and a Pulsar broker, more powerful machines are re ## Install the Pulsar binary package -> You need to install the Pulsar binary package on *each machine in the cluster*, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). +> You need to install the Pulsar binary package on each machine in the cluster, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). Review comment: ```suggestion > You need to install the Pulsar binary package on each machine in the cluster, including machines running ZooKeeper and BookKeeper. ``` ########## File path: site2/docs/deploy-bare-metal.md ########## @@ -76,14 +76,14 @@ For machines running a bookie and a Pulsar broker, more powerful machines are re ## Install the Pulsar binary package -> You need to install the Pulsar binary package on *each machine in the cluster*, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). +> You need to install the Pulsar binary package on each machine in the cluster, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). To get started deploying a Pulsar cluster on bare metal, you need to download a binary tarball release in one of the following ways: * By clicking on the link below directly, which automatically triggers a download: * <a href="pulsar:binary_release_url" download>Pulsar {{pulsar:version}} binary release</a> * From the Pulsar [downloads page](pulsar:download_page_url) -* From the Pulsar [releases page](https://github.com/apache/pulsar/releases/latest) on [GitHub](https://github.com) +* From the Pulsar [releases page](https://github.com/apache/pulsar/releases/latest) on GitHub Review comment: well done~ ########## File path: site2/docs/deploy-bare-metal.md ########## @@ -65,7 +65,7 @@ When you deploy a Pulsar cluster, keep in mind the following basic better choice #### ZooKeeper -For machines running ZooKeeper, is is recommended to use less powerful machines or VMs. Pulsar uses ZooKeeper only for periodic coordination-related and configuration-related tasks, *not* for basic operations. If you run Pulsar on [Amazon Web Services](https://aws.amazon.com/) (AWS), for example, a [t2.small](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/t2-instances.html) instance might likely suffice. +For machines running ZooKeeper, it is recommended to use less powerful machines or VMs. Pulsar uses ZooKeeper only for periodic coordination-related and configuration-related tasks, not for basic operations. If you run Pulsar on [Amazon Web Services](https://aws.amazon.com/) (AWS), for example, a [t2.small](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/t2-instances.html) instance might likely suffice. Review comment: what does "instance might likely suffice" mean? ########## File path: site2/docs/deploy-bare-metal.md ########## @@ -76,14 +76,14 @@ For machines running a bookie and a Pulsar broker, more powerful machines are re ## Install the Pulsar binary package -> You need to install the Pulsar binary package on *each machine in the cluster*, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). +> You need to install the Pulsar binary package on each machine in the cluster, including machines running [ZooKeeper](#deploy-a-zookeeper-cluster) and [BookKeeper](#deploy-a-bookkeeper-cluster). Review comment: we don't need to add link here. Here "ZK" and "BK" are only component, we add link to how to deploy them. It's easily found in the navigation bar. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
