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

Mark Michelson mmichelson at digium.com
Sun Jan 4 15:35:59 CST 2009


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

(Updated 2009-01-04 15:35:59.191792)


Review request for Asterisk Developers and Jared Smith.


Changes
-------

Adding the asterisk-dev group to this review. I'm not sure why it wasn't already there...b


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
-----

  /trunk/CHANGES 167169 
  /trunk/apps/app_queue.c 167169 
  /trunk/main/ast_expr2.h 167169 
  /trunk/main/ast_expr2.c 167169 

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