Skip to content

[Issue #1331] Remove jemalloc prefix in pixels-retina#1332

Merged
bianhq merged 1 commit into
pixelsdb:masterfrom
AntiO2:fix/pixels-retina-jemalloc-no-prefix
Jun 8, 2026
Merged

[Issue #1331] Remove jemalloc prefix in pixels-retina#1332
bianhq merged 1 commit into
pixelsdb:masterfrom
AntiO2:fix/pixels-retina-jemalloc-no-prefix

Conversation

@AntiO2

@AntiO2 AntiO2 commented May 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove --with-jemalloc-prefix=je_ from the bundled jemalloc build in cpp/pixels-retina.
  • Update JNI jemalloc memory metric calls from je_mallctl to mallctl so they match the unprefixed jemalloc build.
  • Keep the optional ENABLE_JEMALLOC=OFF build path compiling.

Validation

  • cmake --build cpp/pixels-retina/build --target pixels-retina -j 4: passed.
  • jemalloc configure output showed JEMALLOC_PREFIX empty.
  • nm -D cpp/pixels-retina/build/jemalloc_install/lib/libjemalloc.so | rg ' (malloc|free|calloc|realloc|mallctl)$| je_(malloc|free|mallctl)$': exported unprefixed malloc, free, calloc, realloc, and mallctl.
  • cmake -S cpp/pixels-retina -B /tmp/pixels-retina-no-jemalloc -DENABLE_JEMALLOC=OFF -DFETCHCONTENT_SOURCE_DIR_GOOGLETEST=/home/antio2/projects/pixels/cpp/pixels-retina/build/_deps/googletest-src: passed.
  • cmake --build /tmp/pixels-retina-no-jemalloc --target pixels-retina -j 4: passed.

Closes #1331

@gengdy1545 gengdy1545 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

@bianhq bianhq merged commit 0081480 into pixelsdb:master Jun 8, 2026
1 check passed
@bianhq bianhq added the enhancement New feature or request label Jun 8, 2026
@bianhq bianhq added this to the Real-time CRUD milestone Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[pixels-retina] Remove jemalloc prefix for LD_PRELOAD allocator interposition

3 participants