Re-enable jemalloc's background thread by default (pingcap#9813) (pin…

…gcap#9821) ref pingcap#9722, close pingcap#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]>
JaySon-Huang · Mar 4, 2025 · 5980ea7 · 5980ea7
1 parent 7985151
commit 5980ea7
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/dbms/src/Server/Server.cpp b/dbms/src/Server/Server.cpp
@@ -448,8 +448,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));