This is an automated email from the ASF dual-hosted git repository.
liaoxin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 3555fd4fc42 [log](load) suppress memtable memory limiter logs (#53277)
3555fd4fc42 is described below
commit 3555fd4fc4203de53e74f2614459f6d43f94db9c
Author: Kaijie Chen <[email protected]>
AuthorDate: Tue Jul 15 21:14:34 2025 +0800
[log](load) suppress memtable memory limiter logs (#53277)
Avoid logging memory limiter messages when the thread wait time is
minimal,
to reduce unnecessary log output and minimize logging overhead.
---
be/src/olap/memtable_memory_limiter.cpp | 22 ++++++++++++----------
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/be/src/olap/memtable_memory_limiter.cpp
b/be/src/olap/memtable_memory_limiter.cpp
index d8efd4e26e1..36e97ce54e5 100644
--- a/be/src/olap/memtable_memory_limiter.cpp
+++ b/be/src/olap/memtable_memory_limiter.cpp
@@ -195,16 +195,18 @@ void
MemTableMemoryLimiter::_handle_memtable_flush(WorkloadGroupPtr wg) {
timer.stop();
int64_t time_ms = timer.elapsed_time() / 1000 / 1000;
g_memtable_memory_limit_latency_ms << time_ms;
- LOG(INFO) << "waited " << PrettyPrinter::print(timer.elapsed_time(),
TUnit::TIME_NS)
- << " for memtable memory limit"
- << ", " <<
GlobalMemoryArbitrator::process_memory_used_details_str() << ", "
- << GlobalMemoryArbitrator::sys_mem_available_details_str()
- << ", load mem: " <<
PrettyPrinter::print_bytes(_mem_tracker->consumption())
- << ", memtable writers num: " << _writers.size()
- << ", active: " << PrettyPrinter::print_bytes(_active_mem_usage)
- << ", queue: " << PrettyPrinter::print_bytes(_queue_mem_usage)
- << ", flush: " << PrettyPrinter::print_bytes(_flush_mem_usage)
- << ", wg: " << (wg ? wg->debug_string() : "null.");
+ if (time_ms > 0) {
+ LOG(INFO) << "waited " << PrettyPrinter::print(timer.elapsed_time(),
TUnit::TIME_NS)
+ << " for memtable memory limit"
+ << ", " <<
GlobalMemoryArbitrator::process_memory_used_details_str() << ", "
+ << GlobalMemoryArbitrator::sys_mem_available_details_str()
+ << ", load mem: " <<
PrettyPrinter::print_bytes(_mem_tracker->consumption())
+ << ", memtable writers num: " << _writers.size()
+ << ", active: " <<
PrettyPrinter::print_bytes(_active_mem_usage)
+ << ", queue: " <<
PrettyPrinter::print_bytes(_queue_mem_usage)
+ << ", flush: " <<
PrettyPrinter::print_bytes(_flush_mem_usage)
+ << ", wg: " << (wg ? wg->debug_string() : "null.");
+ }
}
int64_t MemTableMemoryLimiter::flush_workload_group_memtables(uint64_t wg_id,
int64_t need_flush) {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
