-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3281/
-----------------------------------------------------------
(Updated March 4, 2014, 8:46 p.m.)
Review request for Asterisk Developers.
Changes
-------
Address Matt's feedback.
Bugs: ASTERISK-23204
https://issues.asterisk.org/jira/browse/ASTERISK-23204
Repository: Asterisk
Description
-------
A stasis cache entry now contains more than a single message/snapshot. It
contains messages/snapshots for the internal entity as well as any external
entities that post to the cached item. In addition callbacks can be supplied
when the cache is created to compute and post the aggregate message/snapshot
representing all entities stored in the cache entry.
* All stasis messages now have an eid to indicate what entity posted it.
* The stasis cache enhancements allow device state to cache and aggregate the
device states from internal and external entities in a single operation. The
cached aggregate device state is available immediately after it is posted to
the stasis bus. This improves performance by eliminating a cache dump and
associated ao2 container traversals to calculate the aggregate state.
Diffs (updated)
-----
/trunk/tests/test_stasis.c 409702
/trunk/tests/test_devicestate.c 409702
/trunk/main/stasis_message.c 409702
/trunk/main/stasis_cache.c 409702
/trunk/main/devicestate.c 409702
/trunk/main/app.c 409702
/trunk/include/asterisk/stasis.h 409702
/trunk/include/asterisk/devicestate.h 409702
Diff: https://reviewboard.asterisk.org/r/3281/diff/
Testing
-------
All unit tests pass including the stasis and device state tests.
The device state unit test had to be changed to get the aggregate state out of
the new cache location. Fortunately, the normal users of the device state
aggregate information subscribe to the events and don't get the aggregate
device state out of the cache.
Thanks,
rmudgett
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
