[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16045197#comment-16045197
 ] 

ASF GitHub Bot commented on BOOKKEEPER-1096:
--------------------------------------------

Github user reddycharan commented on the issue:

    https://github.com/apache/bookkeeper/pull/186
  
    @revans2 and @knusbaum FYI, LongHierarchicalLedgerManager needs this fix 
for https://issues.apache.org/jira/browse/BOOKKEEPER-1096


> When ledger is deleted, along with leaf node all the eligible branch nodes 
> also should be deleted in ZooKeeper.
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: BOOKKEEPER-1096
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1096
>             Project: Bookkeeper
>          Issue Type: Improvement
>            Reporter: Charan Reddy Guttapalem
>            Assignee: Charan Reddy Guttapalem
>
> Currently when we delete a ledger, we delete just the leaf node in the ZK but 
> we ignore about the branch nodes. This is ok for FlatLedgerManager, but for 
> HierarchicalLedgerManagers, especially for LongHierarchicalLedgerManager, the 
> number of internal nodes gets blown up over time and we would get into ZK 
> capacity limitations. When ZK reaches the capacity limits, it will manifest 
> in very severe performance and stability issues of cluster. So for 
> HierarchicalLedgerManagers, when we delete a ledger we should optimistically 
> recursively delete the parent znodes as well if they don’t have anymore child 
> znodes. 



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to