Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Redis CPAN distribution.

Report information
The Basics
Id: 64211
Status: resolved
Priority: 0/
Queue: Redis
People
Owner: melo [...] cpan.org
Requestors: mjy [...] geizhals.at
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Dec 25 04:17:04 2010 mjy [...] geizhals.at - Ticket created
Subject: keys() no longer works with Redis 2.0.x
Date: Sat, 25 Dec 2010 10:16:32 +0100
To: bug-Redis [...] rt.cpan.org
From: Marinos Yannikos <mjy [...] geizhals.at>
Apparently the 'keys' command returns *<number of results> as the first response line from 2.x on, Redis.pm expects *<number of bytes in result>. Because of that, keys() returns garbage and the following commands fail.
  Wed Jan 26 09:31:04 2011 GRUBER [...] cpan.org - Correspondence added
On Sat Dec 25 04:17:04 2010, mjy@geizhals.at wrote: Show quoted text
> Apparently the 'keys' command returns *<number of results> as the > first response > line from 2.x on, Redis.pm expects *<number of bytes in result>. > Because of > that, keys() returns garbage and the following commands fail.
I confirm this. A workaround is to remove any special handling of "keys" from the module; then it correctly figures out that it has to use multi-bulk read. So the fix is to just remove lines 133-136.
  Wed Jan 26 09:31:05 2011 The RT System itself - Status changed from 'new' to 'open'
  Sat Feb 05 06:26:16 2011 melo [...] cpan.org - Taken
  Sat Feb 05 06:27:01 2011 melo [...] cpan.org - Correspondence added
Please test version 1.901 from CPAN. It includes full support for Redis 2.x and should fix the problem.
  Sat Feb 05 06:27:01 2011 melo [...] cpan.org - Status changed from 'open' to 'patched'
  Wed May 18 06:28:43 2011 GLORYBOX [...] cpan.org - Correspondence added
Суб Фев 05 06:27:01 2011, MELO писал: Show quoted text
> Please test version 1.901 from CPAN. > > It includes full support for Redis 2.x and should fix the problem.
my @keys = $r->keys('*'); returns some number. version 1.904
  Wed May 18 06:28:44 2011 The RT System itself - Status changed from 'patched' to 'open'
  Wed May 18 08:07:43 2011 GLORYBOX [...] cpan.org - Correspondence added
Срд Май 18 06:28:43 2011, GLORYBOX писал: Show quoted text
> Суб Фев 05 06:27:01 2011, MELO писал:
> > Please test version 1.901 from CPAN. > > > > It includes full support for Redis 2.x and should fix the problem.
> > my @keys = $r->keys('*'); > returns some number. > > version 1.904
I'm sorry, nevermind my comment, please. Everything works as expected.
  Wed May 18 09:15:57 2011 melo [...] cpan.org - Correspondence added
Cools, nice that it works now.
  Wed May 18 09:16:01 2011 melo [...] cpan.org - Status changed from 'open' to 'resolved'