Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

possible EventEmitter memory leak detected #128

Closed
rybesh opened this issue Jan 16, 2016 · 5 comments
Closed

possible EventEmitter memory leak detected #128

rybesh opened this issue Jan 16, 2016 · 5 comments

Comments

@rybesh
Copy link

rybesh commented Jan 16, 2016

I regularly get this warning when running budo:

> baxley@ serve /Users/ryanshaw/Code/react-transcript-player/examples/baxley
> npm -s run watch-css & budo index.js:bundle.js --live | garnish

 info budo: Server running at http://172.31.99.141:9966/ (connect)
 info budo: LiveReload running on 35729
 info browserify: /bundle.js 6580731 2697ms (bundle)
 info http: GET /bundle.js 200 1ms (generated)
(node) warning: possible EventEmitter memory leak detected. 11 readable listeners added. Use emitter.setMaxListeners() to increase limit.
Trace
    at DestroyableTransform.addListener (events.js:239:17)
    at emitDataEvents (/Users/ryanshaw/Code/react-transcript-player/examples/baxley/node_modules/inject-lr-script/node_modules/readable-stream/lib/_stream_readable.js:781:10)
    at DestroyableTransform.Readable.resume (/Users/ryanshaw/Code/react-transcript-player/examples/baxley/node_modules/inject-lr-script/node_modules/readable-stream/lib/_stream_readable.js:755:3)
    at ServerResponse.ondrain (stream.js:41:14)
    at emitNone (events.js:72:20)
    at ServerResponse.emit (events.js:166:7)
    at Socket.ondrain (_http_common.js:193:44)
    at emitNone (events.js:72:20)
    at Socket.emit (events.js:166:7)
    at onwriteDrain (_stream_writable.js:356:12)
@mattdesl
Copy link
Owner

Thanks. Can you update to the latest budo and remove the garnish pipe?

(budo should print 7.1.0 on budo --version)

@mattdesl
Copy link
Owner

I have seen this in a couple issues but I haven't yet been able to reproduce it easily in a test case. It may be related to the server issues or something unrelated.

@quantizor
Copy link

This is a garbage error. You can get rid of it if desired by boosting the max limit. Other variants of EventEmitter (like EE3) have entirely removed the warning since it is nonsensical.

@mattdesl
Copy link
Owner

This should be fixed in 8.0.0 – please let me know if you see it again.

@rybesh
Copy link
Author

rybesh commented Jan 18, 2016

That fixed it; thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants