yiguolei commented on code in PR #50577:
URL: https://github.com/apache/doris/pull/50577#discussion_r2072798005
##########
fe/fe-core/src/main/java/org/apache/doris/resource/workloadgroup/WorkloadGroup.java:
##########
@@ -571,6 +571,15 @@ public void getProcNodeData(BaseProcResult result,
QueryQueue qq) {
row.add(queryQueueDetail == null ? "0" :
String.valueOf(queryQueueDetail.first));
} else if (QueryQueue.WAITING_QUERY_NUM.equals(key)) {
row.add(queryQueueDetail == null ? "0" :
String.valueOf(queryQueueDetail.second));
+ } else if (COMPUTE_GROUP.equals(key)) {
+ String val = properties.get(key);
+ if (Config.isCloudMode()) {
+ String cgName =
Env.getCurrentEnv().getComputeGroupMgr().getComputeGroupNameById(val);
Review Comment:
不要写这种假定的逻辑的代码。
在存算一体的时候，workload group name = workload group id；存算分离的时候，有workload group
name和workload group id，这两个不相等。
所以，我们可以假定workload group 都有id和name，只是存算一体的时候这两者相等。
--
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
