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


##########
be/src/runtime/workload_management/workload_sched_policy_mgr.cpp:
##########
@@ -76,6 +77,7 @@ void WorkloadSchedPolicyMgr::_schedule_workload() {
     while (!_stop_latch.wait_for(std::chrono::milliseconds(500))) {
         // 1 get query info
         std::vector<WorkloadQueryInfo> list;
+        //todo(wb) maybe we can get runtime queryinfo from 
RuntimeQueryStatiticsMgr directly
         _exec_env->fragment_mgr()->get_runtime_query_info(&list);

Review Comment:
   太绕了,直接把这个方法去掉吧, 直接从mgr 里拿到所有的queryid 到 metric map的映射吧



-- 
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