On Thu, Nov 15, 2012 at 10:21 AM, Felix E. Klee <[email protected]> wrote: > Node reports: > > (node) warning: possible EventEmitter memory leak detected. 11 > listeners added. Use emitter.setMaxListeners() to increase limit. > > That there are more than 10 event listeners in my app is no surprise: > For every Socket.IO socket, I set up `n` listeners on "connect" to the > Redis database. The number `n` is around 10 (it corresponds to the > number of certain Redis keys). > > To sum up, with every client (browser) connect, about 10 listeners for > the Redis "connect" event are added. So in the end, there may be > thousands of listeners for the Redis "connect" event. > > I'd like to keep it that way, as the solution appears simple and robust. > > However the above warning makes me suspicious: Is the solution no so > robust after all? Are there potential problems? > > Or could I just do `emitter.setMaxListeners(0)`, and that's it?
Don't read too much into it, it's just a warning. If you have a legitimate reason to add 10+ listeners, go ahead and increase maxListeners. -- Job Board: http://jobs.nodejs.org/ Posting guidelines: https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines You received this message because you are subscribed to the Google Groups "nodejs" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/nodejs?hl=en?hl=en
