em-hiredis icon indicating copy to clipboard operation
em-hiredis copied to clipboard

allow subscribe to multiple channels at once

Open kwojtaszek opened this issue 10 years ago • 3 comments

kwojtaszek avatar Feb 18 '15 20:02 kwojtaszek

Allowing subscribing and unsubscribing to multiple channels seems reasonable since it would then match the design of the redis api (which is the general intent). However I see you've redefined the subscribe method without removing the old implementation, and the new implementation is not api compatible. Thoughts welcome.

Also, psubscribe and punsubscribe should be changed to match if one was to make changes.

Is it worth it?

mloughran avatar Feb 18 '15 23:02 mloughran

@mloughran I didn't want to brake api for subscribing and unsubscribing, that's why add one method that allows user to pass multiple channels to subscribe. Do you think that dropping custom proc is worth breaking the subscription api?

You're right, psubscribe and punsubscribe should be changed to match that.

kwojtaszek avatar Feb 18 '15 23:02 kwojtaszek

:+1:

sponomarev avatar Sep 04 '15 11:09 sponomarev