[ https://issues.apache.org/jira/browse/BOOKKEEPER-1096?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16047897#comment-16047897 ]
Hudson commented on BOOKKEEPER-1096: ------------------------------------ SUCCESS: Integrated in Jenkins build bookkeeper-master #1783 (See [https://builds.apache.org/job/bookkeeper-master/1783/]) BOOKKEEPER-1096: recursive znode delete (sijie: rev 07852d35856dca232450135913090bac27b29abe) * (edit) bookkeeper-server/src/main/java/org/apache/bookkeeper/meta/AbstractZkLedgerManager.java * (add) bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/LedgerMetadataCreationTest.java * (edit) bookkeeper-server/src/test/java/org/apache/bookkeeper/meta/LedgerManagerTestCase.java * (add) bookkeeper-server/src/test/java/org/apache/bookkeeper/util/TestZkUtils.java * (edit) bookkeeper-server/src/main/java/org/apache/bookkeeper/util/ZkUtils.java > 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 > Fix For: 4.5.0 > > > 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.4.14#64029)