Update Redis ascoltatori to use pub_conn instead of client_conn #161

ivanetchart · 2016-11-25T16:49:31Z

No description provided.

mcollina · 2016-11-25T16:57:08Z

lib/redis_ascoltatore.js

@@ -82,7 +82,7 @@ RedisAscoltatore.prototype = Object.create(AbstractAscoltatore.prototype);
 RedisAscoltatore.prototype._startPub = function() {
  var that = this;
  if (this._client === undefined) {
-    this._client = createConn(this._opts, 'client_conn');
+    this._client = createConn(this._opts, 'pub_conn');


can you please make this backward compatible?

coveralls · 2016-11-25T17:00:38Z

Coverage remained the same at 92.778% when pulling 00dda23 on ivanetchart:redis_improvements into a130c66 on mcollina:master.

Update Redis ascoltatori to use pub_conn instead of client_conn

00dda23

mcollina reviewed Nov 25, 2016

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update Redis ascoltatori to use pub_conn instead of client_conn #161

Update Redis ascoltatori to use pub_conn instead of client_conn #161

ivanetchart commented Nov 25, 2016

mcollina Nov 25, 2016

coveralls commented Nov 25, 2016 •

edited

Loading

Update Redis ascoltatori to use pub_conn instead of client_conn #161

Are you sure you want to change the base?

Update Redis ascoltatori to use pub_conn instead of client_conn #161

Conversation

ivanetchart commented Nov 25, 2016

mcollina Nov 25, 2016

Choose a reason for hiding this comment

coveralls commented Nov 25, 2016 • edited Loading

coveralls commented Nov 25, 2016 •

edited

Loading