xinyiZzz commented on code in PR #50107:
URL: https://github.com/apache/doris/pull/50107#discussion_r2051465576
##########
be/src/runtime/runtime_query_statistics_mgr.cpp:
##########
@@ -521,4 +521,12 @@ void
RuntimeQueryStatisticsMgr::get_active_be_tasks_block(vectorized::Block* blo
}
}
+void RuntimeQueryStatisticsMgr::get_tasks_resource_context(
Review Comment:
我也考虑过这个，想要拿到所有任务的 resource ctx，两个选择
1. RuntimeQueryStatisticsMgr，现成的
2. WorkloadGroupMgr，遍历所有 workload group 拿到所有任务的 resource context。
目前 active tasks 这个系统表由 RuntimeQueryStatisticsMgr 产出，既然
RuntimeQueryStatisticsMgr 保存的是 resource ctx，未来 RuntimeQueryStatisticsMgr
是否考虑包含所有类型的任务？
TODO， RuntimeQueryStatisticsMgr 目前保存的 shared ptr resource ctx，应该改成 weak ptr
--
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]
