Skip to content

balancerd: enable jemalloc#28056

Merged
benesch merged 1 commit intoMaterializeInc:mainfrom
benesch:balancer-jemalloc
Jul 8, 2024
Merged

balancerd: enable jemalloc#28056
benesch merged 1 commit intoMaterializeInc:mainfrom
benesch:balancer-jemalloc

Conversation

@benesch
Copy link
Copy Markdown
Contributor

@benesch benesch commented Jul 6, 2024

Bring balancerd into alignment with our other production binaries and use jemalloc by default on Linux.

Motivation

  • This PR refactors existing code.

Checklist

Bring balancerd into alignment with our other production binaries and
use jemalloc by default on Linux.
@benesch benesch requested a review from madelynnblue July 6, 2024 18:20
@benesch benesch merged commit 456b883 into MaterializeInc:main Jul 8, 2024
def- added a commit to def-/materialize that referenced this pull request May 7, 2026
Since MaterializeInc#28056, `main()` registered jemalloc/alloc metrics into one
MetricsRegistry while `run()` created a separate one for the HTTP
`/metrics` endpoint, making the alloc metrics invisible. Pass the
registry from `main()` into `run()` to match environmentd and clusterd.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants