<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/3227/">https://reviewboard.asterisk.org/r/3227/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I have some general-purpose notes here:
1) Since realtime queues are a configuration replacement, I think that the scripts adding realtime queues should go in the config/ directory instead of its own new directory.
2) This is the first time I've seen booleans used in the alembic script. From a programming perspective, this seems like the correct thing to do. What I'm curious about is whether this is a good idea since most of our documentation shows a yes/no value instead of a true/false value for these options. Since booleans aren't being used in other alembic scripts, their use here is odd.
3) Other alembic scripts have created enums for options that have a limited set of values. This same practice can be applied to certain queue options. For instance, the strategy option could use an enum to limit the choices to the valid ring strategies.</pre>
<br />
<p>- Mark Michelson</p>
<br />
<p>On February 15th, 2014, 1:40 a.m. UTC, 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 Feb. 15, 2014, 1:40 a.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/ASTERISK-23233">ASTERISK-23233</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;">Created the queues, queue_members, and cdr alembic scripts.
The CDR table is more of an example for new setups since the actual table can be fully customized in cdr_adaptive_odbc.conf.</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;">Created a postgres DB to see that queues and members were read in.</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/12/contrib/ast-db-manage/queues/versions/14b983d1bb27_create_queue_tables.py <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/queues/script.py.mako <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/queues/env.py <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/queues.ini.sample <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/cdr/versions/210693f3123d_create_cdr_table.py <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/cdr/script.py.mako <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/cdr/env.py <span style="color: grey">(PRE-CREATION)</span></li>
<li>/branches/12/contrib/ast-db-manage/cdr.ini.sample <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3227/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>