[MemTracker][Bug-Fix] Fix core in DECHECK in memory tracker (#4421)
Fix DECHECK failed in mem_tracker, issue #4420
This commit is contained in:
@ -58,7 +58,7 @@ public:
|
||||
: _mgr(mgr),
|
||||
_state(state),
|
||||
_tracker(tracker),
|
||||
_query_tracker(MemTracker::CreateTracker(-1, "BufferedBlockMgr2", _mgr->_mem_tracker->parent())),
|
||||
_query_tracker(_mgr->_mem_tracker->parent()),
|
||||
_num_reserved_buffers(num_reserved_buffers),
|
||||
_num_tmp_reserved_buffers(0),
|
||||
_num_pinned_buffers(0) {
|
||||
|
||||
@ -479,7 +479,7 @@ class MemTracker : public std::enable_shared_from_this<MemTracker> {
|
||||
DCHECK(consumption_metric_ == nullptr) << "Should not be called on root.";
|
||||
for (MemTracker* tracker : all_trackers_) {
|
||||
if (tracker == end_tracker) return;
|
||||
DCHECK(!tracker->has_limit());
|
||||
DCHECK(!tracker->has_limit()) << tracker->label() << " have limit:" << tracker->limit();
|
||||
tracker->consumption_->add(bytes);
|
||||
}
|
||||
DCHECK(false) << "end_tracker is not an ancestor";
|
||||
|
||||
Reference in New Issue
Block a user