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

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2020/04/20 22:54:12 UTC

[GitHub] [bookkeeper] mino181295 commented on issue #2316: API to delete multiple ledgers at once

mino181295 commented on issue #2316:
URL: https://github.com/apache/bookkeeper/issues/2316#issuecomment-616853216


   I can take this, a few question i had looking around the code:
   
   - Is it better to convert 
   `removeLedgerMetadata(long ledgerId, Version version)`
   to: `removeLedgerMetadata(Set<Long> ledgerIds, Version version)`
   Or do you suggest creating a brand new method?
   - In HierarchicalLedgerManager zk deletions are performed recursively and optimistically. Shoul i create a `ZKUtils#asyncCreateFullPathOptimistic` similar that runs multi instead of delete recursively?
   


----------------------------------------------------------------
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:
users@infra.apache.org