Skip to content

json-serde: streaming md5#382

Open
mateusz834 wants to merge 2 commits into
hanabi1224:mainfrom
mateusz834:jsonserde-zig
Open

json-serde: streaming md5#382
mateusz834 wants to merge 2 commits into
hanabi1224:mainfrom
mateusz834:jsonserde-zig

Conversation

@mateusz834

@mateusz834 mateusz834 commented Apr 5, 2023

Copy link
Copy Markdown

(for canada 100)

Before:

User time (seconds): 3.61
Maximum resident set size (kbytes): 542020

After:

User time (seconds): 3.45
Maximum resident set size (kbytes): 114816
@hanabi1224

Copy link
Copy Markdown
Owner

Thanks for the PR! It's a great optimization, but I think it makes the zig solution incomparable to others, I might need more time to think about it.

@mateusz834

mateusz834 commented Apr 13, 2023

Copy link
Copy Markdown
Author

How about 2-streaming.zig ? I also added streaming md5 for go and rust. (go doesn't have any performance improvements, it buffers the entire json before writing) and added error handling for the go code.

@mateusz834 mateusz834 changed the title json-serde zig: streaming md5 Apr 13, 2023
@hanabi1224 hanabi1224 force-pushed the main branch 3 times, most recently from 3fc55d3 to d70acc7 Compare July 12, 2023 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants