search

taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.15k stars 131 forks source link

[#113] Parse database number out of redis uri #140

Closed olek closed 8 years ago

olek commented 8 years ago

Here is my take on fixing #113.

Currently any information in redis uri path is simply ignored. Summary of this pull request: if path is a non-default db number, it is extracted and used, otherwise it is ignored.

ptaoussanis commented 8 years ago

Thanks Olek, looks good!

ptaoussanis commented 8 years ago

Have cut a private release for this as [com.taoensso/carmine "2.12.0-beta1"].

olek commented 8 years ago

Thank you Peter for such quick turnaround! My first ever OSS Clojure PR accepted. Weee!

And another thank you for private release. Now I can remove that questionable rails-style 'monkey patch' from our project.

ptaoussanis commented 8 years ago

No problem Olek :-) Happy Clojure-ing!