jaredhanson / electrolyte

Elegant dependency injection for Node.js.
MIT License
564 stars 61 forks source link

Performance? #6

Closed niftylettuce closed 9 years ago

niftylettuce commented 10 years ago

Not sure if this has been tested before – but are there any ways we can optimize this?

jaredhanson commented 10 years ago

Is there any concerns specifically? What areas are performance critical or potentially performing sub-optimally?

dkushner commented 9 years ago

@niftylettuce, I second @jaredhanson's question. Have you noticed some performance issues while using electrolyte in production? I'm currently contemplating a move to electrolyte because it just seems so damn clean, simple and structured. However, if that comes at a great cost I'd reconsider.

jaredhanson commented 9 years ago

I'm going to close this if there performance degradation can't be demonstrated. I'm using electrolyte extensively in all my applications and haven't noticed adverse effects.

niftylettuce commented 9 years ago

@dkushner no significant performance issues, i just wanted someone to run tests and put some metrics somewhere