wangbo commented on code in PR #53688:
URL: https://github.com/apache/doris/pull/53688#discussion_r2228181360
##########
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:
这行删了算是默认行为变更，得确认下所有调这个的地方是否有依赖这行代码的
--
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]
