Skip Menu |

This queue is for tickets about the Test-Mock-Redis CPAN distribution.

Report information
The Basics
Id: 116659
Status: new
Priority: 0/
Queue: Test-Mock-Redis

People
Owner: Nobody in particular
Requestors: JRR [...] cpan.org
Cc: brian.lucas [...] heartit.com
AdminCc:

Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)



CC: brian.lucas [...] heartit.com
Subject: Support expiry & existential tests in SET command
The current implementation of SET does not take into account the additional features introduced in Redis 2.6: Show quoted text
> Starting with Redis 2.6.12 SET supports a set of options that modify its behavior: > - EX seconds -- Set the specified expire time, in seconds. > - PX milliseconds -- Set the specified expire time, in milliseconds. > - NX -- Only set the key if it does not already exist. > - XX -- Only set the key if it already exist.
I am happy to write some tests and a patch for these new behaviors, but I'm unsure where to put them. I'd probably put the NX/XX behaviors in 06-keys.t. Do you think the EX/PX ones belong there as well, or perhaps in 07-expires.t? Cheers, Jordan Reuter PAUSE: JRR
Ha, I should have looked at the GitHub PRs before opening this ticket. On Tue Aug 02 12:29:09 2016, JRR wrote: Show quoted text
> The current implementation of SET does not take into account the > additional features introduced in Redis 2.6: >
> > Starting with Redis 2.6.12 SET supports a set of options that modify > > its behavior: > > - EX seconds -- Set the specified expire time, in seconds. > > - PX milliseconds -- Set the specified expire time, in > > milliseconds. > > - NX -- Only set the key if it does not already exist. > > - XX -- Only set the key if it already exist.
> > I am happy to write some tests and a patch for these new behaviors, > but I'm unsure where to put them. I'd probably put the NX/XX behaviors > in 06-keys.t. Do you think the EX/PX ones belong there as well, or > perhaps in 07-expires.t? > > Cheers, > Jordan Reuter > > PAUSE: JRR