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.

Sign up for GitHub

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

Jump to bottom

Benchmarks vs OTP 27 json module #27

Closed
axelson opened this issue Dec 6, 2024 · 4 comments · May be fixed by michalmuskala/jason#198
Closed

Benchmarks vs OTP 27 json module #27

axelson opened this issue Dec 6, 2024 · 4 comments · May be fixed by michalmuskala/jason#198

Comments

@axelson
Copy link

axelson commented Dec 6, 2024

Hi, is it possible to benchmark jsonrs vs the new json module that's been added in OTP 27? I'm curious how they'll stack up!
@dlindenkreuz dlindenkreuz mentioned this issue Dec 20, 2024
Open
@dlindenkreuz
Copy link

I just added Jsonrs and the new Elixir.JSON module to the benchmark suite to the Jason repository: michalmuskala/jason#198

Screenshots from a quick run on my machine (json refers to Erlang's json module, available since OTP 27):
Elixir's new JSON module adds a negligible overhead to Erlang's json which it calls under the hood.

Screenshot 2024-12-20 at 11 27 42 Screenshot 2024-12-20 at 11 27 53

@axelson
Copy link
Author

axelson commented Dec 20, 2024

Nice! Thank you for running them. Is the benchmark script somewhere in the git repo? I'm having trouble finding it.

@dlindenkreuz
Copy link

It’s not in this repo but in the Jason repo: https://github.com/michalmuskala/jason/tree/f1aa12ba6949dd90f6b45519d13699a3e9829f55/bench

(link leads to my PR branch)

@axelson
Copy link
Author

axelson commented Dec 21, 2024

Ah, got it. Thank you!

@axelson axelson closed this as completed Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update benchmarks dlindenkreuz/jason
2 participants
@axelson @dlindenkreuz