[asterisk-dev] [Code Review] New queue CLI and Manager commands to facilitate fine-grained reloading

Mark Michelson mmichelson at digium.com
Wed Feb 11 14:19:50 CST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/115/
-----------------------------------------------------------

(Updated 2009-02-11 14:19:50.900203)


Review request for Asterisk Developers and Jared Smith.


Changes
-------

Get rid of irrelevant changes from the diff.


Summary
-------

The purpose of these changes is to add new CLI and manager commands in order to reload queues.

The new CLI commands are the "queue reload" and "queue reset stats" commands. The new manager commands
are the QueueReload QueueReset commands. Rather than explain them in detail here, I'll direct you to the
modified documentation in the code.

In order to easily facilitate these changes, I also revamped the backend functions. Now instead of having
one giant function to reload queues, there are multiple small ones.


Diffs (updated)
-----

  /trunk/CHANGES 174883 
  /trunk/apps/app_queue.c 174883 

Diff: http://reviewboard.digium.com/r/115/diff


Testing
-------

I ran many combinations of the CLI commands in succession, and a lesser number of the Manager commands. All appeared to do what is expected.


Thanks,

Mark




More information about the asterisk-dev mailing list