yiguolei commented on code in PR #53688:
URL: https://github.com/apache/doris/pull/53688#discussion_r2228418789
##########
fe/fe-core/src/main/java/org/apache/doris/resource/workloadgroup/WorkloadGroupMgr.java:
##########
@@ -179,22 +132,16 @@ public WorkloadGroup
getWorkloadGroupByComputeGroup(WorkloadGroupKey wgKey) {
return keyToWorkloadGroup.get(wgKey);
}
- public List<TPipelineWorkloadGroup> getWorkloadGroup(ConnectContext
context) throws UserException {
+ // the workload group name in AllBackendComputeGroup will associate with
multiple workload groups
+ public List<WorkloadGroup> getWorkloadGroup(ConnectContext context) throws
UserException {
String wgName = getWorkloadGroupNameAndCheckPriv(context);
ComputeGroup cg = context.getComputeGroup();
-
- List<TPipelineWorkloadGroup> workloadGroups = Lists.newArrayList();
readLock();
try {
- List<WorkloadGroup> wgList = cg.getWorkloadGroup(wgName, this);
- for (WorkloadGroup wg : wgList) {
- workloadGroups.add(wg.toThrift());
- }
- context.setWorkloadGroupName(wgName);
Review Comment:
这个只有审计日志里用，我在coordinator那里显示的设置了这个了
--
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]
