[BOOKKEEPER-529] stopServingSubscriber in delivery manager should remove stub callbacks in ReadAheadCache

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.2.0
• Component/s: hedwig-server
• Assignee: Sijie Guo
• Reporter: Sijie Guo
• Created: 29/Dec/12 00:50
• Updated: 13/Feb/13 15:46
• Resolved: 09/Jan/13 16:47

Description

Currently each subscriber would insert stub callback to wait newly published messages or scanning result. for waiting scan result, it was OK. the callback would be triggered and removed after scan callback arrived. but if it was wait newly published, it would be a problem. if sub/closesub/sub become frequent, closesub doesn't remove the installed callback, so the stub callbacks are accumulated, which cause the memory increased, finally OOM.

it would be better to remove its installed stub callback when closesub.

Issue Links

• blocks: BOOKKEEPER-539 ClientNotSubscribedException & doesn't receive enough messages in TestThrottlingDelivery#testServerSideThrottle