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

JSON.stringify #27

Open
FlynnLeeGit opened this issue Jan 28, 2017 · 5 comments
Open

JSON.stringify #27

FlynnLeeGit opened this issue Jan 28, 2017 · 5 comments

Comments

@FlynnLeeGit
Copy link

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

@StarpTech
Copy link

@FlynnLeeGit read his post https://community.risingstack.com/the-worlds-fastest-javascript-memoization-library/ he compares different serializer.

@caiogondim
Copy link
Owner

@FlynnLeeGit you have data to prove your point?
Please consider creating a benchmark similar to the ones on the benchmark/ folder.

Opinions without data are, well, just opinions =)

@Frondor
Copy link

Frondor commented Sep 23, 2018

What about using https://github.com/fastify/fast-json-stringify ?

@planttheidea
Copy link
Collaborator

@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.

@caiogondim
Copy link
Owner

@Frondor That would be cool to have a benchmark with that lib. Can you do a PR?

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

5 participants