sijie commented on issue #1336: Use metadata service uri rather than zkServers URL: https://github.com/apache/bookkeeper/pull/1336#issuecomment-380638114 NOTES: This change looks changes a lot. But it mostly touches test cases and does straightforward one-one mapping change, should be fairly straightforward to review. conf.getZkServers => ZkMetadataDriverBase.resolveZkServers(conf) conf.getZkLedgersRootPath => ZkMetadataDriverBase.resolveZkLedgersRootPath(conf) conf.setZkServers + conf.getZkLedgersRootPath => conf.setMetadataServiceUri("zk://" + zkServers + "/ledgers")
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services