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

[NSE-433]Support Pre-built Jemalloc #434

Closed
wants to merge 4 commits into from

Conversation

weiting-chen
Copy link
Collaborator

What changes were proposed in this pull request?

To build Jemalloc v5.2.1 from source.

How was this patch tested?

Pass the test in the local server.

@github-actions
Copy link

#433

@weiting-chen
Copy link
Collaborator Author

I notice my PR includes another PR for k8s support.
Will drop this PR and commit a new one for Jemalloc and another one for K8s.

Copy link
Collaborator

@zhouyuan zhouyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weiting-chen thanks for the work, we need to modify codes to use this new allocator

"--disable-shared"
"--disable-cxx"
"--disable-libdl"
"--disable-initial-exec-tls"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add one prefix here: --with-jemalloc-prefix=je_gazelle_

@weiting-chen
Copy link
Collaborator Author

Will re-submit a new PR to remote K8s code and address this issue only.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants