[GitHub] [pulsar] Jennifer88huang commented on a change in pull request #5164: [doc] Improve Pulsar Deployment monitoring

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

Jennifer88huang commented on a change in pull request #5164: [doc] Improve Pulsar Deployment monitoring
URL: https://github.com/apache/pulsar/pull/5164#discussion_r322565717
 
 

 ##########
 File path: site2/docs/deploy-monitoring.md
 ##########
 @@ -35,60 +34,52 @@ http://$BROKER_ADDRESS:8080/metrics
 
 ### ZooKeeper stats
 
-The local Zookeeper/configuration store server and clients that are shipped with Pulsar have been instrumented to expose
-detailed stats through Prometheus as well.
+The local Zookeeper and configuration store server and clients that are shipped with Pulsar have been instrumented to expose detailed stats through Prometheus as well.
 
 ```shell
 http://$LOCAL_ZK_SERVER:8000/metrics
 http://$GLOBAL_ZK_SERVER:8001/metrics
 ```
 
-The default port of local ZooKeeper is `8000` and that of configuration store is `8001`.
-These can be changed by specifying system property `stats_server_port`.
+The default port of local ZooKeeper is `8000` and the default port of configuration store is `8001`. You can change the default port of local Zookeeper and configuration store by specifying system property `stats_server_port`.
 
 ### BookKeeper stats
 
 For BookKeeper you can configure the stats frameworks by changing the `statsProviderClass` in
 `conf/bookkeeper.conf`.
 
-By default, the default BookKeeper configuration included with Pulsar distribution will enable
-the Prometheus exporter.
+By default, the default BookKeeper configuration included with Pulsar distribution enables the Prometheus exporter.
 
 ```shell
 http://$BOOKIE_ADDRESS:8000/metrics
 ```
 
-For bookies, the default port is `8000` (instead of `8080`) and that can be configured by changing
-the `prometheusStatsHttpPort` in `conf/bookkeeper.conf`.
+For bookies, the default port is `8000` (instead of `8080`) and you can configure the default port of bookies by changing the `prometheusStatsHttpPort` in `conf/bookkeeper.conf`.
 
 Review comment:
   You can refine the sentence further.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services