You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@bookkeeper.apache.org by "gaozhangmin (via GitHub)" <gi...@apache.org> on 2026/07/13 08:54:45 UTC

Re: [PR] Issue 4800: Reuse metadata driver for over-replicated ledger GC [bookkeeper]

gaozhangmin commented on PR #4801:
URL: https://github.com/apache/bookkeeper/pull/4801#issuecomment-4956166502

   > Thanks for improving this path and avoiding repeated metadata driver initialization.
   > 
   > One thing that might be worth considering is whether we can wire in the bookie server’s existing MetadataBookieDriver or LedgerManagerFactory here. Since the server already creates and owns those resources through its lifecycle, GC could potentially borrow the factory and only create/close the per-run LedgerUnderreplicationManager. That would keep the ownership/close semantics centralized while still achieving the reuse goal.
   
   PTAL
   


-- 
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@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org