gaozhangmin commented on pull request #14686: URL: https://github.com/apache/pulsar/pull/14686#issuecomment-1072457817
> > This pr is aimed to solve failed initialized cluster metadata. > > This makes sense to me. but if we merge this and we don't do the other changes in any case users won't be able to run Pulsar with this configuration. So this patch by itself is useless. Without this change, 1、neither `initialize-cluster-metadata --metadata-store zk:zk1:2181,zk2:2181/test` nor `initialize-cluster-metadata -metadata-store zk:zk1:2181;zk2:2181/test` supported. 2、 neither set `metadataServiceUri="metadata-store:zk:zk1:2181,zk2:2181/test"` nor `metadataServiceUri="metadata-store:zk:zk1:2181;zk2:2181/test"` in bookkeeper.conf supported. That's what problem this pr tries to solve. other changes try to support setting `metadataServiceUri="metadata-store:zk:zk1:2181,zk2:2181/test"` in bookkeeper.conf. @eolivelli -- 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]
