yiguolei commented on code in PR #34729:
URL: https://github.com/apache/doris/pull/34729#discussion_r1597466292


##########
be/src/pipeline/pipeline_task.cpp:
##########
@@ -315,7 +315,8 @@ bool PipelineTask::should_revoke_memory(RuntimeState* 
state, int64_t revocable_m
     wg->check_mem_used(&is_wg_mem_low_water_mark, &is_wg_mem_high_water_mark);
     if (is_wg_mem_high_water_mark) {
         if (revocable_mem_bytes > min_revocable_mem_bytes) {
-            LOG_EVERY_N(INFO, 10) << "revoke memory, hight water mark";
+            LOG_EVERY_T(INFO, 1) << "query " << print_id(state->query_id())

Review Comment:
   Not print this log, it should be a debug log. 
   This method is called by pipeline engine during every loop.
   If there is many queries, the log is too often.



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to