[asterisk-bugs] [Asterisk 0012858]: "queue show" shows old (cached) information, whilst "queue show QUEUENAME" shows current information when using realtime.

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12858 
====================================================================== 
Reported By:                bcnit
Assigned To:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   12858
Category:                   Applications/app_queue
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.0-beta9 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             06-14-2008 07:54 CDT
Last Modified:              06-26-2008 15:50 CDT
====================================================================== 
Summary:                    "queue show" shows old (cached) information, whilst
"queue show QUEUENAME" shows current information when using realtime.
Description: 
Whenever 'queue show' is used, realtime is not queried for any updates.
When 'queue show QUEUENAME' is used, it is.

This is defined in apps/app_queue.c:

/* We only want to load realtime queues when a specific queue is asked
for. */
if (argc == 3)  /* specific queue */
   load_realtime_queue(argv[2]);

I appreciate that this has been deliberately coded this way, but in my
opinion, this is erroneous behaviour.

If realtime information is not loaded for each queue, then the output of
'queue show' is just plain wrong and can't be relied on - the command
simply becomes useless.

The response to bug http://bugs.digium.com/view.php?id=0012556 was that the AMI
should be used to collect
status information. At the moment, one must be aware of each queue name in
order to retrieve status information for it - The problem is that third
party applications cannot report on the status of queues using the AMI
without having this knowledge (i.e. access to the realtime data store). In
addition, even if they were aware of this, they would have to make one
'queue show QUEUENAME' call for each of the queues they require information
on - this is clearly inefficient at best.

I believe that the fact that 'queue show' returns incorrect information
should be considered a bug.

====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 06-26-08 15:50  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 125477

U   trunk/apps/app_queue.c

------------------------------------------------------------------------
r125477 | mmichelson | 2008-06-26 15:50:29 -0500 (Thu, 26 Jun 2008) | 19
lines

Merged revisions 125476 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r125476 | mmichelson | 2008-06-26 15:56:01 -0500 (Thu, 26 Jun 2008) | 11
lines

Prior to this patch, the "queue show" command used cached
information for realtime queues instead of giving up-to-date
info. Now realtime is queried for the latest and greatest in
queue info.

(closes issue http://bugs.digium.com/view.php?id=12858)
Reported by: bcnit
Patches:
      queue_show.patch uploaded by putnopvut (license 60)


........

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

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
06-26-08 15:50  svnbot         Checkin                                      
06-26-08 15:50  svnbot         Note Added: 0089313                          
======================================================================