[BOOKKEEPER-1096] When ledger is deleted, along with leaf node all the eligible branch nodes also should be deleted in ZooKeeper.

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.5.0
• Assignee: Charan Reddy Guttapalem
• Reporter: Charan Reddy Guttapalem
• Created: 09/Jun/17 23:04
• Updated: 13/Jun/17 13:26
• Resolved: 12/Jun/17 15:21

Description

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.

Issue Links

• links to: GitHub Pull Request #186