[Home]

Summary:ASTERISK-12471: [patch] Rename rrmemory strategy to roundrobin
Reporter:Eliel Sardanons (eliel)Labels:
Date Opened:2008-07-28 12:12:18Date Closed:2008-07-30 14:09:22
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_queue.diff
Description:According to UPGRADE-1.4.txt 'rrmemory' will be renamed to 'roundrobin' in 1.6.
Here is the patch for that.

Patch uploaded against trunk but applies to 1.6.0.
Comments:By: Mark Michelson (mmichelson) 2008-07-28 12:27:04

Hmm, I see the note in UPGRADE-1.4.txt that you refer to, but I think that renaming "rrmemory" to "roundrobin" at this point would be very confusing. I think that what would be less confusing (and probably would get less people annoyed at us) would be to allow the rrmemory strategy to be described as either "rrmemory" or "roundrobin."

I can check with kpfleming first, though, since he was the original author of that note in UPGRADE-1.4.txt.
By: Eliel Sardanons (eliel) 2008-07-28 13:56:49

I agree with you.
By: Digium Subversion (svnbot) 2008-07-30 14:09:20

Repository: asterisk
Revision: 134483

U   trunk/apps/app_queue.c

------------------------------------------------------------------------
r134483 | tilghman | 2008-07-30 14:09:19 -0500 (Wed, 30 Jul 2008) | 4 lines

Let "roundrobin" also reference rrmemory, for the 1.6 release (as described in
UPGRADE-1.4.txt)
(Closes issue ASTERISK-12471)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=134483