-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3114/#review10571
-----------------------------------------------------------



/branches/11/main/cli.c
<https://reviewboard.asterisk.org/r/3114/#comment20035>

    This should explain what "silent" means if it is specified.


- Mark Michelson


On Jan. 10, 2014, 7:26 p.m., rmudgett wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3114/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2014, 7:26 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: AST-1252
>     https://issues.asterisk.org/jira/browse/AST-1252
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> The per console verbose level feature as previously implemented caused a 
> large performance penalty.  The fix required some minor incompatibilities if 
> the new rasterisk is used to connect to an earlier version.  If the new 
> rasterisk connects to an older Asterisk version then the root console verbose 
> level is always affected by the "core set verbose" command of the remote 
> console even though it may appear to only affect the current console.  If an 
> older version of rasterisk connects to the new version then the "core set 
> verbose" command will have no effect.
> 
> * Fixed the verbose performance by not generating a verbose message if 
> nothing is going to use it and then filtered any generated verbose messages 
> before actually sending them to the remote consoles.
> 
> * Split the "core set debug" and "core set verbose" CLI commands to remove 
> the per module verbose support that cannot work with the per console verbose 
> level.
> 
> * Added a silent option to the "core set verbose" command.
> 
> * Fixed "core set debug off" tab completion.
> 
> * Made "core show settings" list the current console verbosity in addition to 
> the root console verbosity.
> 
> * Changed the default verbose level of the 'verbose' setting in the 
> logger.conf [logfiles] section.  The default is now to once again follow the 
> current root console level.  As a result, using the AMI Command action with 
> "core set verbose" could again set the root console verbose level and affect 
> the verbose level logged.
> 
> 
> Diffs
> -----
> 
>   /branches/11/main/pbx.c 405296 
>   /branches/11/main/manager.c 405296 
>   /branches/11/main/logger.c 405296 
>   /branches/11/main/cli.c 405296 
>   /branches/11/main/asterisk.c 405296 
>   /branches/11/include/asterisk/logger.h 405296 
>   /branches/11/funcs/func_timeout.c 405296 
>   /branches/11/configs/logger.conf.sample 405296 
>   /branches/11/apps/app_verbose.c 405296 
>   /branches/11/apps/app_dumpchan.c 405296 
>   /branches/11/UPGRADE.txt 405296 
> 
> Diff: https://reviewboard.asterisk.org/r/3114/diff/
> 
> 
> Testing
> -------
> 
> Setup an extension that had a simple dialplan extension that decremented a 
> counter from 500000 and nothing logging or viewing verbose messages.
> On my machine, Asterisk v11 without the patch took 63 seconds while with the 
> patch it took 25 seconds.
> 
> Tested the "core set verbose" CLI command tab completion.
> 
> Tested that different verbose levels on multiple rasterisk connections got 
> the expected verbose messages.
> 
> 
> 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

Reply via email to