xinyiZzz commented on code in PR #48293:
URL: https://github.com/apache/doris/pull/48293#discussion_r1969104961
##########
be/src/runtime/fragment_mgr.cpp:
##########
@@ -909,21 +911,21 @@ Status FragmentMgr::_get_query_ctx(const Params& params,
TUniqueId query_id, boo
_exec_env->runtime_query_statistics_mgr()->set_workload_group_id(
print_id(query_id), tg_id);
- LOG(INFO) << "Query/load id: " <<
print_id(query_ctx->query_id())
- << ", use workload group: "
- << workload_group_ptr->debug_string()
- << ", is pipeline: " <<
((int)is_pipeline);
+ VLOG(10) << "Query/load id: " <<
print_id(query_ctx->query_id())
+ << ", use workload group: "
+ << workload_group_ptr->debug_string()
+ << ", is pipeline: " <<
((int)is_pipeline);
} else {
- LOG(INFO) << "Query/load id: " <<
print_id(query_ctx->query_id())
- << " carried group info but can not find
group in be";
+ VLOG(10) << "Query/load id: " <<
print_id(query_ctx->query_id())
+ << " carried group info but can not find
group in be";
}
}
// There is some logic in query ctx's dctor, we could not
check if exists and delete the
// temp query ctx now. For example, the query id maybe
removed from workload group's queryset.
map.insert({query_id, query_ctx});
- LOG(INFO) << "Register query/load memory tracker,
query/load id: "
- << print_id(query_ctx->query_id()) << " limit: "
- << PrettyPrinter::print(query_ctx->mem_limit(),
TUnit::BYTES);
+ VLOG(10) << "Register query/load memory tracker,
query/load id: "
Review Comment:
这行日志和上面 869 行保留一个就行，可以在 869 行增加打印 `mem_limit`，然后把这行删了
--
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
