Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

Bug fix and additional feature. #7

Closed
wants to merge 2 commits into from

Conversation

mjssjmgithub
Copy link

I found an problem when issuing subsequent subscribe commands on a PubSub connection already handling incoming messages from Redis . Also, I added the ability to register a ChannelHander to a pipeline. Since Redis subscriptions are bound to the Redis connection it is useful to know if the connection was severed. This means all subscriptions on that connection are gone and messages may have been missed.

@wg
Copy link
Owner

wg commented Mar 2, 2013

Hello Matt, thank you for debugging this issue! I'll be shortly pushing a very slightly different fix and releasing a new version that includes it.

I'm wary of making the public lettuce API dependent on netty, so I'll look into other options for notifying interested parties of connection loss. It might be something best added to the RedisPubSubListener interface.

@wg wg closed this Mar 2, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants