Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Support jemalloc (optionally) #1100

Closed
pjenvey opened this issue Dec 19, 2017 · 0 comments
Closed

Support jemalloc (optionally) #1100

pjenvey opened this issue Dec 19, 2017 · 0 comments
Assignees
@pjenvey
Labels
enhancement Operational
Milestone
2017Q4-2.1 PyPy m...

Comments

@pjenvey
Copy link
Member

pjenvey commented Dec 19, 2017

Given our excessive memory usage under glibc malloc push-service#78 we should support easily swapping it out for jemalloc.

It's known to be a lot more consistent in RSS size vs glibc malloc in a number of situations.

We should be able to inject it via an LD_PRELOAD. The Docker file can ensure it's installed and handle setting the LD_PRELOAD appropriately depending on some external environment variable. This way it can be easily toggled off/on.
@pjenvey pjenvey self-assigned this Dec 19, 2017
@pjenvey pjenvey mentioned this issue Dec 19, 2017
Closed
pjenvey added a commit that referenced this issue Jan 5, 2018
@pjenvey
feat: allow optional use of jemalloc
3b1c72a 
via USE_JEMALLOC which defaults to false

Closes #1100
@pjenvey pjenvey mentioned this issue Jan 6, 2018
Merged
pjenvey added a commit that referenced this issue Jan 8, 2018
@pjenvey
feat: allow optional use of jemalloc
0ba6603 
via USE_JEMALLOC which defaults to false

Closes #1100
pjenvey added a commit that referenced this issue Jan 22, 2018
@pjenvey
feat: allow optional use of jemalloc
99b3345 
via USE_JEMALLOC which defaults to false

Closes #1100

(cherry picked from commit 0ba6603)
@pjenvey pjenvey mentioned this issue Mar 5, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pjenvey pjenvey

Labels
enhancement Operational
Projects
None yet
Milestone
2017Q4-2.1 PyPy memory allocation and...
Development

No branches or pull requests
2 participants
@pjenvey @bbangert