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

New one that focused on performance. #303

Open
ifsnow opened this issue Nov 11, 2016 · 0 comments
Open

New one that focused on performance. #303

ifsnow opened this issue Nov 11, 2016 · 0 comments

Comments

@ifsnow
Copy link

ifsnow commented Nov 11, 2016

First of all, thank you for giving me an opportunity to do something fun.

I'm impressed with the concept of co module.
so, I tried making a new implementation that focused on performance.
I think it's worth doing such a micro-optimization. Not all, but at least in the core modules.

Here's my working branch.

These are my goals.

* Run faster.

  • benchmark - co() / test code

    • Node 4.4.7 : 8% faster
    • Node 6.9.1 : 12% faster
  • benchmark - co.wrap() / test code

    • Node 4.4.7 : 3% faster
    • Node 6.9.1 : 8% faster
  • benchmark - koa / test code

    • Node 4.4.7 : 15% faster
    • Node 6.9.1 : 32% faster

* Keep compatibility.

It's fully compatibility with original co. Passed all test cases.


I'm thinking of providing it for the public with a different module. for example, co-booster.
but, I'm not sure it would be helpful to them as much as I think. :)

Please let me know if you have any concerns or advice.

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

1 participant