BewareMyPower commented on pull request #8269:
URL: https://github.com/apache/pulsar/pull/8269#issuecomment-709323654
LGTM. Just one point, I think it's better to change
> --existing-bk-metadata-service-uri "zk+null://bk1:2181;bk2:2181/ledgers"
to `zk1:2181,zk2:2181` to emphasize that the metadata store is ZK (not
Etcd,etc.) or just `host1:2181,host2:2181` to be consistent with other params.
----------------------------------------------------------------
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:
[email protected]
