<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/3114/">https://reviewboard.asterisk.org/r/3114/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Performance speedup is as expected; no other regressions found.</pre>
<br />
<p>- Guenther Kelleter</p>
<br />
<p>On January 10th, 2014, 8:26 p.m. CET, rmudgett wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By rmudgett.</div>
<p style="color: grey;"><i>Updated Jan. 10, 2014, 8:26 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/AST-1252">AST-1252</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/11/main/pbx.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/main/manager.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/main/logger.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/main/cli.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/main/asterisk.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/include/asterisk/logger.h <span style="color: grey">(405296)</span></li>
<li>/branches/11/funcs/func_timeout.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/configs/logger.conf.sample <span style="color: grey">(405296)</span></li>
<li>/branches/11/apps/app_verbose.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/apps/app_dumpchan.c <span style="color: grey">(405296)</span></li>
<li>/branches/11/UPGRADE.txt <span style="color: grey">(405296)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3114/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>