[Asterisk-code-review] stasis cache: Don't cache stasis subscription change messages (asterisk[13])

George Joseph asteriskteam at digium.com
Fri Aug 31 07:51:56 CDT 2018


Hello Jenkins2, Corey Farrell, Joshua Colp, 

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

    https://gerrit.asterisk.org/10023

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

Change subject: stasis_cache: Don't cache stasis_subscription_change messages
......................................................................

stasis_cache: Don't cache stasis_subscription_change messages

stasis_subscription_change messages can provide useful information
to topic subscribers but caching them makes no sense since they
don't alter the state of the object the topic is related to.
They do however take CPU cycles to manage in the cache and consume
memory in the cache container.

ASTERISK-27121

Change-Id: I3d18905e477f3721815da91f30da8d3fbb2d4f56
---
M main/stasis_cache.c
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/23/10023/2
-- 
To view, visit https://gerrit.asterisk.org/10023
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3d18905e477f3721815da91f30da8d3fbb2d4f56
Gerrit-Change-Number: 10023
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180831/344b639e/attachment.html>


More information about the asterisk-code-review mailing list