ans76 commented on a change in pull request #6979:
URL: https://github.com/apache/incubator-doris/pull/6979#discussion_r740071851
##########
File path: be/src/runtime/mem_tracker.h
##########
@@ -170,6 +172,12 @@ class MemTracker : public
std::enable_shared_from_this<MemTracker> {
Release(-bytes);
return true;
}
+ if (MemInfo::current_mem() + bytes >= MemInfo::mem_limit()) {
+ VLOG_RPC << "TryConsume failed, bytes=" << bytes
+ << " process whole consumption=" << MemInfo::current_mem()
+ << " mem limit=" << limit << " attempting to GC";
Review comment:
That's may logging not accurate memory usage for that moment if set
`refresh_current_memory_usage_interval_ms` longer? we experienced memory usage
increasing fast in seconds.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
