Hi Steve,<br><br>If only you had been more helpful in your first e-mail!<br><br>I&#39;ve already taken care of this, but thanks!<br><br>Sean<br><br><div><span class="gmail_quote">On 7/18/07, <b class="gmail_sendername">Steve Totaro
</b> &lt;<a href="mailto:stotaro@totarotechnologies.com">stotaro@totarotechnologies.com</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Sean,<br><br>I see what you mean.&nbsp;&nbsp;Maybe this will help?<br><br>Consider what would happen in a large scale deployment with hundreds of<br>agents and hundreds more callers.&nbsp;&nbsp;All the agents are on calls and there<br>is no delay in retry.&nbsp;&nbsp;It sounds like the equivalent to a very tight
<br>loop or packet storm and does not sound like a good idea.<br><br>Try removing or commenting the retry entry and see if the one second<br>delay is still an issue.<br><br>* apps/app_queue.c: Setting a retry of 0 is generally not a good
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;idea and shouldn&#39;t be allowed. (#7574 - reported by regin)<br><br><a href="http://bugs.digium.com/view.php?id=7574">http://bugs.digium.com/view.php?id=7574</a><br><br>Hope that helps.&nbsp;&nbsp;It looks like maybe B.J
. Weschke could clarify this<br>more if needed since he is the one that introduced the code that does<br>not allow 0.<br><br>Thanks,<br>Steve Totaro<br><br>Maybe in a<br><br>Sean Bright wrote:<br>&gt; Hi Steve,<br>&gt;<br>
&gt; Thank you for the tips, but I think you may have misunderstood my<br>&gt; original e-mail.<br>&gt;<br>&gt; My original question had nothing to do with not liking the rationale<br>&gt; behind why that design decision was made, but simply not actually
<br>&gt; _knowing_ the reason behind that decision.<br>&gt;<br>&gt; Obviously, I have already changed app_queue to allow the &#39;retry&#39; value<br>&gt; to be 0, and have tested it successfully in my local development<br>
&gt; environment (If you&#39;d like a patch against trunk or one of the release<br>&gt; branches, I would be happy to share.&nbsp;&nbsp;Its just a one-liner.).&nbsp;&nbsp;So far,<br>&gt; I have experienced no notable side effects.&nbsp;&nbsp;That being said, it _is_
<br>&gt; simply a development environment that isn&#39;t under any significant<br>&gt; load, so I can&#39;t be sure without testing in a production environment.<br>&gt;<br>&gt; And that is the reason I asked here.&nbsp;&nbsp;There was a reason that the
<br>&gt; original author decided not to allow &#39;retry&#39; values of 0, and that may<br>&gt; be because it causes problems elsewhere in the PBX.&nbsp;&nbsp;That is the<br>&gt; information I was after, so I can avoid pushing code into production
<br>&gt; that might bring the PBX down.<br>&gt;<br>&gt; Once I get that information (or I am at least satisfied that my change<br>&gt; will not adversely affect the production environment) I will be happy<br>&gt; to submit a patch to the bug tracker.&nbsp;&nbsp;My disclaimer has been on file
<br>&gt; for months, so that won&#39;t be a problem.<br>&gt;<br>&gt; Thanks again for your response!<br>&gt; Sean<br>&gt;<br>&gt; On 7/18/07, *Steve Totaro* &lt;<a href="mailto:stotaro@totarotechnologies.com">stotaro@totarotechnologies.com
</a><br>&gt; &lt;mailto:<a href="mailto:stotaro@totarotechnologies.com">stotaro@totarotechnologies.com</a>&gt;&gt; wrote:<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; Sean,<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; The beauty of open source software is the ability to change it.
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; If you<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; do not like the &quot;rationale&quot;, you can sign a disclaimer and submit a<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; patch to bugtracker for consideration of inclusion.<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; I am sure you will get much more feedback in that forum from
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; developers<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; and bug marshals (unless of course they just close it with no<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; explanation ;-)<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; Thanks,<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; Steve Totaro<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; Sean Bright wrote:<br>
&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; Awesome.&nbsp;&nbsp;Thanks a bunch.<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; On 7/17/07, *Sean Bright* &lt;<a href="mailto:sean.bright@gmail.com">sean.bright@gmail.com</a><br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &lt;mailto:<a href="mailto:sean.bright@gmail.com">
sean.bright@gmail.com</a>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; &lt;mailto:<a href="mailto:sean.bright@gmail.com">sean.bright@gmail.com</a> &lt;mailto:<a href="mailto:sean.bright@gmail.com">sean.bright@gmail.com</a>&gt;&gt;&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; wrote:
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; Hey guys,<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; I know this is the &quot;wrong&quot; list, but I&#39;m more interested in the<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; rationale behind this decision...<br>
&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; Why is the &#39;retry&#39; value in queues.conf limited to values &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; 0?&nbsp;&nbsp;I<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; am using rrmemory with a queue, and I am forced to wait at<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; least 1
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; second between attempts to contact the next agent in<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; line.&nbsp;&nbsp;I can<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; take this question to the -users list if necessary, but I doubt<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; I&#39;ll get a satisfactory response there.
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; Thanks,<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; Sean<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; ------------------------------------------------------------------------<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; _______________________________________________<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; --Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--">http://www.api-digital.com--</a><br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; asterisk-dev mailing list
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt; To UNSUBSCRIBE or update options visit:<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; &gt;&nbsp;&nbsp;&nbsp;&nbsp; <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br>&gt;<br>&gt;<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; _______________________________________________
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; --Bandwidth and Colocation Provided by<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; <a href="http://www.api-digital.com--">http://www.api-digital.com--</a> &lt;<a href="http://www.api-digital.com--">http://www.api-digital.com--</a>&gt;<br>&gt;
<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; asterisk-dev mailing list<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp; To UNSUBSCRIBE or update options visit:<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev
</a><br>&gt;<br>&gt;<br>&gt; ------------------------------------------------------------------------<br>&gt;<br>&gt; _______________________________________________<br>&gt; --Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--">
http://www.api-digital.com--</a><br>&gt;<br>&gt; asterisk-dev mailing list<br>&gt; To UNSUBSCRIBE or update options visit:<br>&gt;&nbsp;&nbsp;&nbsp;&nbsp;<a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev
</a><br><br><br>_______________________________________________<br>--Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--">http://www.api-digital.com--</a><br><br>asterisk-dev mailing list<br>To UNSUBSCRIBE or update options visit:
<br>&nbsp;&nbsp; <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br>