-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for Redis database #2
Conversation
…ller and implemented RedisController for use with Redis.
Thanks! The code looks good so far. The only thing that is a little unclean is the handling of the For the Redis implementation, there are now two new modules, |
Ok, I'll look into types and idioms end of week as I have time. |
The redis idioms looks promising and I'm trying to migrate my code to it or at least be forward compatible. So far I've stumbled over two issues: 1. PagingHave you already considered implementing sorted collections in redis idioms? What about storing the IDs The
2. Object relationshipsThe |
|
I'll merge it like it is for now and try to make any breaking database related changes before tagging the next release. The only thing that I'm still thinking about is how to best go about handling IDs without producing allocations - probably using a fixed size struct that is able to hold a |
Add support for Redis database.
As discussed in issue #1 I implemented Redis support by generalizing and abstracting the UserManController and implementing db specific subclasses for both mongodb and redis.
Breaking changes:
UserManSettings
has now adatabaseURL
property replacing thedatabaseName
.User
andGroup
properties have changed as follows:BsonObjectID _id
is nowstring id
resetCodeExpireTime
is nowSysTime
instead ofBsonDate
.fromBson
andtoBson
methods have been added to account for this when serializing to BSON.