[GitHub] [pulsar] DaveDuggins commented on a diff in pull request #16159: [improve][docs] Use metadataServiceUri in favor of zkServers

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

DaveDuggins commented on code in PR #16159:
URL: https://github.com/apache/pulsar/pull/16159#discussion_r959627014


##########
conf/bookkeeper.conf:
##########
@@ -631,10 +631,10 @@ diskCheckInterval=10000
 ## Metadata Service settings
 #############################################################################
 
-# metadata service uri that bookkeeper is used for loading corresponding metadata driver and resolving its metadata service location
+# Metadata service uri that bookkeeper uses for loading the corresponding metadata driver and resolving its metadata service location
 # Examples: 
-#  - metadataServiceUri=metadata-store:zk:my-zk-1:2181
-#  - metadataServiceUri=metadata-store:etcd:http://my-etcd:2379
+#  - metadataServiceUri=zk://my-zk-1:2181/ledgers
+#  - metadataServiceUri=etcd:http://my-etcd:2379

Review Comment:
   > Hi @DaveDuggins Does this look ok
   
   Yes. Anonymitaet pinged me for reference. I'm not the approver. We still need a committer to merge this.



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