athanatos opened a new pull request #1669: ISSUE 1662: LocalBookKeeper use layout manager and ledger path configs URL: https://github.com/apache/bookkeeper/pull/1669 The URI patch had a side effect of causing LocalBookKeeper to always use a URI string of the form zk:// which defaults to HLM. This patch updates LocalBookKeeper to generate zk+<layout>:// based on the legacy LEDGER_MANAGER_FACTORY_CLASS config when present. +null will be used otherwise and defaults to HLM. Further, use the (admittedly deprecated) zkLedgerRootPath config to determine the ledger path (defaults to /ledger, so the behavior will be unchanged if unspecified). (@bug W-5415276@) (@rev cguttapalem@) Signed-off-by: Samuel Just <[email protected]>
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
