[asterisk-dev] [Code Review]: Pinequeue: Play queue prompts in the background - making call available to agent faster

Leif Madsen reviewboard at asterisk.org
Sun Apr 29 09:11:09 CDT 2012



> On April 26, 2012, 11:37 a.m., jrose wrote:
> >
> 
> Leif Madsen wrote:
>     And further to the note, is there really no way to have such dependency on the announcement length having such an adverse effect to all the queue positions being announced? That is a really awkward way of having to handle things. I can imagine someone wanting to adjust the announcements dynamically through the Record() application so that they can update announcements, hold times etc...
>     
>     I really don't like the dependency on the file length and the adjustment of the periodic announce values to provide a sane experience to the caller.
> 
> Olle E Johansson wrote:
>     Thanks for the feedback, Leif.
>     
>     If you can find funding I'm sure we can find a way. For this customer, this is acceptable. And I'd rather document this than just let it slip by. Isn't that better?
>     
>     /O

I disagree that's better. Doing it right and giving the general community an experience that makes sense is important. There are so many things in Asterisk that were little one offs to fix a particular persons issue and then not cleaned up prior to merge, that you end up having to document around things, or the experience continuing changes over and over again. I think Asterisk has gone far enough that we should be doing it right, not just doing it for the sake of doing it.

I personally would not be comfortable with the feature going in as-is. If someone else in the community needs to finish the feature up so it works in a manner that many will expect, then so be it. If you need to continue supporting this as-is out of band for your customer, then so be that as well.


- Leif


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


On April 26, 2012, 9:42 a.m., Olle E Johansson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1887/
> -----------------------------------------------------------
> 
> (Updated April 26, 2012, 9:42 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Today, when a prompt is being played to a call waiting in the queue and an agent becomes available, the agent will not get the call until the prompt is finished. Both customer and agent is kept waiting.
> 
> With this rather big piece of code, we attach a generator to play prompts. The generator can, like all generators, be stopped at any time so that the agent (queue member) can get the call immediately.
> 
> The generator is now placed in app_queue, but could propably be moved somewhere else. It also changes functionality in main/say.c in order to be able to place those prompts in the same prompt queue for background processing. The same generator could be used to sevice conference bridges and maybe be added as a dialplan function at some point for background playlists.
> 
> 
> This addresses bug 19795.
>     https://issues.asterisk.org/jira/browse/19795
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_queue.c 364000 
>   /trunk/configs/queues.conf.sample 364000 
>   /trunk/include/asterisk/channel.h 364000 
>   /trunk/include/asterisk/file.h 364000 
>   /trunk/main/asterisk.dynamics 364000 
>   /trunk/main/file.c 364000 
>   /trunk/main/say.c 364000 
> 
> Diff: https://reviewboard.asterisk.org/r/1887/diff
> 
> 
> Testing
> -------
> 
> Quite a lot of testing in our environment during the rather long time we've been testing this. Customer is happy.
> 
> 
> There was some complaints from a bowl of petunias, which made me a bit surprised.
> 
> 
> Thanks,
> 
> Olle E
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120429/56770fb6/attachment.htm>


More information about the asterisk-dev mailing list