5

xinyiZzz · Aug 7, 2024 · 6baff5a · 6baff5a
1 parent ecb952d
commit 6baff5a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/be/src/common/config.cpp b/be/src/common/config.cpp
@@ -139,7 +139,7 @@ DEFINE_mBool(enable_stacktrace, "true");
 
 DEFINE_mInt64(stacktrace_in_alloc_large_memory_bytes, "2147483648");
 
-DEFINE_mBool(enable_memory_orphan_check, "false");
+DEFINE_mBool(enable_memory_orphan_check, "true");
 
 // The maximum time a thread waits for full GC. Currently only query will wait for full gc.
 DEFINE_mInt32(thread_wait_gc_max_milliseconds, "1000");

diff --git a/be/src/vec/common/allocator.cpp b/be/src/vec/common/allocator.cpp
@@ -215,7 +215,7 @@ void Allocator<clear_memory_, mmap_populate, use_mmap, MemoryAllocator>::memory_
 template <bool clear_memory_, bool mmap_populate, bool use_mmap, typename MemoryAllocator>
 void Allocator<clear_memory_, mmap_populate, use_mmap, MemoryAllocator>::consume_memory(
         size_t size) {
-    DCHECK(tracker->label() == doris::thread_context()->thread_mem_tracker()->label())
+    DCHECK(doris::thread_context()->thread_mem_tracker()->label() == tracker->label())
             << ", thread mem tracker label: "
             << doris::thread_context()->thread_mem_tracker()->label()
             << ", allocator tracker label: " << tracker->label();
@@ -227,7 +227,7 @@ void Allocator<clear_memory_, mmap_populate, use_mmap, MemoryAllocator>::release
         size_t size) const {
     doris::ThreadContext* thread_context = doris::thread_context(true);
     if (thread_context && thread_context->thread_mem_tracker()->label() != "Orphan") {
-        DCHECK(tracker->label() == thread_context->thread_mem_tracker()->label())
+        DCHECK(thread_context->thread_mem_tracker()->label() == tracker->label())
                 << ", thread mem tracker label: " << thread_context->thread_mem_tracker()->label()
                 << ", allocator tracker label: " << tracker->label();
         RELEASE_THREAD_MEM_TRACKER(size);