-
Notifications
You must be signed in to change notification settings - Fork 86
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
JSON.stringify #27
Comments
@FlynnLeeGit read his post https://community.risingstack.com/the-worlds-fastest-javascript-memoization-library/ he compares different serializer. |
@FlynnLeeGit you have data to prove your point? Opinions without data are, well, just opinions =) |
What about using https://github.com/fastify/fast-json-stringify ? |
@FlynnLeeGit, just add an object version of the same benchmark that exists. It will surprise how much the performance drops when real serialization needs to occur. |
@Frondor That would be cool to have a benchmark with that lib. Can you do a PR? |
I think Json.stringify is too slow when arguments has big list object,it will take a long time to parse from object to string,in this situation, new Map() is a 10x faster than Json.stringify,maybe can change the way which strategy to choose by arguments types
The text was updated successfully, but these errors were encountered: