Add support to optionally use redislite

kiddouk / redisco

A Python Library for Simple Models and Containers Persisted in Redis

MIT License

439 stars 78 forks source link

Add support to optionally use redislite #43

Closed dwighthubbard closed 8 years ago

prologic commented 9 years ago

:+1: I'm defintely going to use this :)

prologic commented 9 years ago

Ignore my last silly comment :-) I'm merging your fork into mine and testing this out

prologic commented 9 years ago

So :+1: it works really nicely :)

prologic commented 9 years ago

Hey @dwighthubbard I tried out your patch/pull-request in my branch and I found an issue that I'd like to address:

We need a way to be explicit about whether we use redis or redislite here.

Right now I think you use Redislite if it's "importable". The use-case I want to have for https://github.com/prologic/charla is:

charla --embedded

Makes sense? :)

dwighthubbard commented 9 years ago

Redislite will pass the request through to the Redis module if the user passes a hostname or socketfile of a Redis server as an argument. This was done in order to prevent the need to write code in every app to fail back to Redis if the user doesn't want to use the embedded server.