-
Notifications
You must be signed in to change notification settings - Fork 716
Closed
Labels
C-dependenciesCategory: Pull requests that update a dependency fileCategory: Pull requests that update a dependency fileC-good-first-issueCategory: issues that are self-contained and easy for newcomers to work on.Category: issues that are self-contained and easy for newcomers to work on.NodeNode teamNode teamT-nodeTeam: issues relevant to the node experience teamTeam: issues relevant to the node experience team
Description
We are currently using https://github.com/gnzlbg/jemallocator. We should switch to https://github.com/tikv/jemallocator.
See rust-lang/rust#83152 for context.
My TL;DR: first crate uses old jemalloc, and it doesn't set it up to use the most efficient APIs. tikv fork fixes both problems, and delivers a sizable performance increase. I've just re-checked it, and for one rust-analyzer this yields 5% decrease in the number of executed instructions.
Metadata
Metadata
Assignees
Labels
C-dependenciesCategory: Pull requests that update a dependency fileCategory: Pull requests that update a dependency fileC-good-first-issueCategory: issues that are self-contained and easy for newcomers to work on.Category: issues that are self-contained and easy for newcomers to work on.NodeNode teamNode teamT-nodeTeam: issues relevant to the node experience teamTeam: issues relevant to the node experience team