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: 73297
Status: resolved
Priority: 0/
Queue: Redis
People
Owner: Nobody in particular
Requestors: jeff [...] lanndark.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Dec 15 17:25:44 2011 jeff [...] lanndark.com - Ticket created
Subject: Feature: Timeout support
Date: Thu, 15 Dec 2011 14:25:33 -0800
To: bug-Redis [...] rt.cpan.org
From: Jeff <jeff [...] lanndark.com>
It would be nice to allow a specified client timeout value for redis operations. If the server is taking too long to respond, I don't really want my process waiting around. I believe this could be as simple as a one-line change to pass a timeout value through into the IO::Socket::INET constructor, something like: $self->{sock} = IO::Socket::INET->new( PeerAddr => $self->{server}, Proto => 'tcp', Timeout => $self->{timeout} || undef # I think undef is the current behavior - untested. ) || confess...
  Fri Dec 16 13:36:12 2011 melo [...] simplicidade.org - Correspondence added
Subject: Re: [rt.cpan.org #73297] Feature: Timeout support
Date: Fri, 16 Dec 2011 18:35:42 +0000
To: bug-Redis [...] rt.cpan.org
From: Pedro Melo <melo [...] simplicidade.org>
Hi, yes, you should have something like that (with optional auto-reconnect) soon. Bye, On Thu, Dec 15, 2011 at 10:25 PM, Jeff via RT <bug-Redis@rt.cpan.org> wrote: Show quoted text
> Thu Dec 15 17:25:44 2011: Request 73297 was acted upon. > Transaction: Ticket created by jeff@lanndark.com >       Queue: Redis >     Subject: Feature: Timeout support >   Broken in: (no value) >    Severity: (no value) >       Owner: Nobody >  Requestors: jeff@lanndark.com >      Status: new >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=73297 > > > > It would be nice to allow a specified client timeout value for redis > operations. If the server is taking too long to respond, I don't really > want my process waiting around. > > I believe this could be as simple as a one-line change to pass a timeout > value through into the IO::Socket::INET constructor, something like: > > > $self->{sock} = IO::Socket::INET->new( >  PeerAddr => $self->{server}, >  Proto => 'tcp', >  Timeout => $self->{timeout} || undef     # I think undef is the current > behavior - untested. > ) || confess... >
-- Pedro Melo @pedromelo http://www.simplicidade.org/ http://about.me/melo xmpp:melo@simplicidade.org mailto:melo@simplicidade.org
  Fri Dec 16 13:36:14 2011 The RT System itself - Status changed from 'new' to 'open'
  Tue Mar 06 04:31:01 2012 melo [...] cpan.org - Correspondence added
Hi, Implemented in the latest release. Thanks -- Hi, how are you?
  Tue Mar 06 04:31:03 2012 melo [...] cpan.org - Status changed from 'open' to 'resolved'