Skip to content

[parity-util-mem] Consider using tikv-jemallocator instead of jemallocator #586

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
Closed
ordian opened this issue Sep 3, 2021 · 2 comments · Fixed by #589
Closed

[parity-util-mem] Consider using tikv-jemallocator instead of jemallocator #586

ordian opened this issue Sep 3, 2021 · 2 comments · Fixed by #589
Assignees
@koute
Labels
F6-refactor M5-dependencies

Comments

@ordian
Copy link
Member

ordian commented Sep 3, 2021

rust-lang/rust#83152
https://docs.rs/tikv-jemallocator
rust-rocksdb/rust-rocksdb#554
@koute
Copy link
Contributor

koute commented Sep 13, 2021

Good idea; I'll try to run some basic performance/memory tests and push a PR.

@koute koute self-assigned this Sep 13, 2021
@koute
Copy link
Contributor

koute commented Sep 14, 2021

From very rough tests it looks like the CPU usage is mostly unchanged, but the peak RSS memory usage is around ~10% lower, so should be worth migrating to!

@koute koute mentioned this issue Sep 20, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@koute koute

Labels
F6-refactor M5-dependencies
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Switch from jemallocator to tikv-jemallocator koute/parity-common
2 participants
@koute @ordian