Added support for SETEX

alphazero / jredis

Java Client and Connectors for Redis

http://code.google.com/p/jredis/

Apache License 2.0

316 stars 136 forks source link

Added support for SETEX #33

Closed jlavallee closed 13 years ago

jlavallee commented 14 years ago

Howdy, I've added support for SETEX in my fork on the master branch.

If there are additional tests you'd like to see please just let me know.

Thanks!

Jeff

alphazero commented 14 years ago

Hi Jeff. Will try to get to all pull request over the weekend. Thanks for contributing.

alphazero commented 14 years ago

Just reviewed your changes. Perfect. If you want to add treatment for JRedisFuture, go ahead and I'll put the whole set. /R

jlavallee commented 14 years ago

Thanks for reviewing the changes so quickly! I don't know when I might have time to monkey with the futures bit, so you probably may as well pull this bit by itself. If I find some more time to hack on JRedis I'll be sure to send another pull request your way :)

Thanks,

Jeff