This is an automated email from the ASF dual-hosted git repository.

yiguolei pushed a commit to branch dev-1.1.2
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/dev-1.1.2 by this push:
     new c33ebbf03f [dev-1.1.2](memtracker) Fix DCHECK consumption is greater 
than 0 in old mem tracker #11980
c33ebbf03f is described below

commit c33ebbf03f7a54886a5b3280d5fd92cae2cc0bd8
Author: Xinyi Zou <zouxiny...@gmail.com>
AuthorDate: Tue Aug 23 11:00:52 2022 +0800

    [dev-1.1.2](memtracker) Fix DCHECK consumption is greater than 0 in old mem 
tracker #11980
---
 be/src/runtime/mem_tracker.cpp |  4 ++--
 be/src/runtime/mem_tracker.h   | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/be/src/runtime/mem_tracker.cpp b/be/src/runtime/mem_tracker.cpp
index 350f7bc311..d3779f8dbb 100644
--- a/be/src/runtime/mem_tracker.cpp
+++ b/be/src/runtime/mem_tracker.cpp
@@ -266,8 +266,8 @@ MemTracker::~MemTracker() {
     delete reservation_counters_.load();
 
     if (parent()) {
-        DCHECK(consumption() == 0) << "Memory tracker " << debug_string()
-                                   << " has unreleased consumption " << 
consumption();
+        // DCHECK(consumption() == 0) << "Memory tracker " << debug_string()
+        //                            << " has unreleased consumption " << 
consumption();
         parent_->Release(consumption());
 
         lock_guard<SpinLock> l(parent_->child_trackers_lock_);
diff --git a/be/src/runtime/mem_tracker.h b/be/src/runtime/mem_tracker.h
index 9166462aba..8b9f754633 100644
--- a/be/src/runtime/mem_tracker.h
+++ b/be/src/runtime/mem_tracker.h
@@ -249,11 +249,11 @@ public:
             /// metric. Don't blow up in this case. (Note that this doesn't 
affect non-process
             /// trackers since we can enforce that the reported memory usage 
is internally
             /// consistent.)
-            if (LIKELY(tracker->consumption_metric_ == nullptr)) {
-                DCHECK_GE(tracker->consumption_->current_value(), 0)
-                        << std::endl
-                        << tracker->LogUsage(UNLIMITED_DEPTH);
-            }
+            // if (LIKELY(tracker->consumption_metric_ == nullptr)) {
+            //     DCHECK_GE(tracker->consumption_->current_value(), 0)
+            //             << std::endl
+            //             << tracker->LogUsage(UNLIMITED_DEPTH);
+            // }
         }
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to