Zend Framework

Zend_Queue_Adapter_Memcacheq::isExists($name) not work

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.10.5
  • Fix Version/s: None
  • Component/s: Zend_Queue
  • Labels:
    None

Description

error in http://framework.zend.com/code/browse/Zend_Framework/standard/trunk/library/Zend/Queue/Adapter/Memcacheq.php?r=20096#l221

Memcacheq return:

stats queue
STAT queue_name1 66/66
STAT queue_name2 21/21
STAT queue_name3 14/14

http://memcachedb.org/memcacheq/ has error.
see section "General-purpose statistics" on http://memcachedb.googlecode.com/svn/trunk/doc/protocol.txt

Activity

Hide
Tim de Pater - Enrise added a comment -

The output of the stats command is change in version 0.2.0, see: http://www.serverphorums.com/read.php?9,74777,92986
I've fixed this with a regular expression, not sure if this is the best solution but it works and is backwards compatible

Line 221:

$this->_queues[] = preg_replace('~STAT (\w+)(?:[ 0-9]*/?[0-9]*)~', '\\1', $line);
Show
Tim de Pater - Enrise added a comment - The output of the stats command is change in version 0.2.0, see: http://www.serverphorums.com/read.php?9,74777,92986 I've fixed this with a regular expression, not sure if this is the best solution but it works and is backwards compatible Line 221:
$this->_queues[] = preg_replace('~STAT (\w+)(?:[ 0-9]*/?[0-9]*)~', '\\1', $line);
Hide
Tim de Pater - Enrise added a comment -

I've a patch attached to fix this issue, slightly changed the regex I already posted.
Can someone review this to make sure it's working correctly?

Show
Tim de Pater - Enrise added a comment - I've a patch attached to fix this issue, slightly changed the regex I already posted. Can someone review this to make sure it's working correctly?
Hide
Dolf Schimmel (Freeaqingme) added a comment -

Tim, could you please provide a couple of new possible values (or even better, unittests), so we can actually test this issue in the testsuite?

Thanks.

Show
Dolf Schimmel (Freeaqingme) added a comment - Tim, could you please provide a couple of new possible values (or even better, unittests), so we can actually test this issue in the testsuite? Thanks.

People

Vote (2)
Watch (3)

Dates

  • Created:
    Updated: