[Asterisk-code-review] app_voicemail: Remove dependency on the stasis cache (...asterisk[master])

Kevin Harwell asteriskteam at digium.com
Tue Jul 9 09:36:11 CDT 2019


Hello Benjamin Keith Ford, Friendly Automation, George Joseph, Joshua Colp, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/11466

to look at the new patch set (#7).

Change subject: app_voicemail: Remove dependency on the stasis cache
......................................................................

app_voicemail: Remove dependency on the stasis cache

app_voicemail utilized the stasis cache when polling mailboxes for MWI. This
caused a memory leak (items were not being appropriately removed from the
cache), and subsequent slowdown in system processing. This patch removes the
stasis cache dependency, thus alleviating the memory leak. It does this by
utilizing the new MWI API that better manages state lifetime.

ASTERISK-28443
ASTERISK-27121

Change-Id: Ie89fedaca81ea1fd03d150d9d3a1ef3d53740e46
---
M apps/app_voicemail.c
1 file changed, 90 insertions(+), 221 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/66/11466/7
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/11466
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ie89fedaca81ea1fd03d150d9d3a1ef3d53740e46
Gerrit-Change-Number: 11466
Gerrit-PatchSet: 7
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190709/6c05c4e5/attachment.html>


More information about the asterisk-code-review mailing list