[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