Re-enable jemalloc's background thread by default (#9813) (#9821)

ref #9722, close #9812 Re-enable jemalloc's background thread by default Signed-off-by: ti-chi-bot <[email protected]> Signed-off-by: JaySon-Huang <[email protected]> Co-authored-by: Calvin Neo <[email protected]> Co-authored-by: JaySon-Huang <[email protected]> Co-authored-by: JaySon <[email protected]>
pingcap · Feb 11, 2025 · d8f0c00 · d8f0c00
1 parent 7d751d6
commit d8f0c00
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/dbms/src/Server/Server.cpp b/dbms/src/Server/Server.cpp
@@ -489,8 +489,10 @@ void UpdateMallocConfig([[maybe_unused]] const LoggerPtr & log)
 
     LOG_INFO(log, "Got jemalloc config: opt.background_thread {}, opt.max_background_threads {}", old_b, old_max_thd);
 
-    if (!malloc_conf && !old_b)
+    bool not_config_bg = !malloc_conf || strstr(malloc_conf, "background_thread") == nullptr;
+    if (not_config_bg && !old_b)
     {
+        // If the user doesn't explicitly set the background_thread opt, and it is actually false, then set it to true.
         LOG_INFO(log, "Try to use background_thread of jemalloc to handle purging asynchronously");
 
         RUN_FAIL_RETURN(je_mallctl("max_background_threads", nullptr, nullptr, (void *)&new_max_thd, sz_st));