search

killme2008 / carmine-sentinel

A Clojure library designed to connect redis by sentinel, make carmine to support sentinel.
Eclipse Public License 1.0
14 stars 4 forks source link

支持 with-new-pubsub-listener #16

Closed ylgrgyq closed 5 years ago

killme2008 commented 5 years ago

Update the readme?

ylgrgyq commented 5 years ago

哎呀，才注意到 Readme 里有对 pubsub, message queue 和 lock 的支持说明。有点纠结还有没有必要像这个 PR 这样封一下，还是就让用户去直接使用 update-conn-spec ？ 只改造一下 pubsub 不管 message queue 和 lock 感觉有点不统一，像是漏掉了这两个功能一样。 把他们都改造一下都放在 carmine-sentinel 里又感觉是让 carmine-sentinel 做的事情变得不单一，把几个不同功能强行拉到一起。 现在看上去直接使用 update-conn-spec 好像也不是特别讨厌 😂 。你感觉呢？是保持原状，直接使用 update-conn-spec，还是把 message queue, lock 都像 pubsub 这样改一下，还是只改 pubsub 留下 message queue 和 lock ?