wangbo commented on code in PR #49884: URL: https://github.com/apache/doris/pull/49884#discussion_r2036399795
########## fe/fe-core/src/main/java/org/apache/doris/resource/workloadgroup/WorkloadGroupMgr.java: ########## @@ -200,82 +203,78 @@ private void writeUnlock() { lock.writeLock().unlock(); } - public List<TPipelineWorkloadGroup> getWorkloadGroup(ConnectContext context) throws UserException { - String groupName = getWorkloadGroupNameAndCheckPriv(context); + private WorkloadGroup getWorkloadGroupByComputeGroupUnlock(Pair<String, String> key) + throws DdlException { + WorkloadGroup wg = nameToWorkloadGroup.get(key); + if (wg == null) { + throw new DdlException("can not find workload group " + key.second + " in " + key.first + "."); + } + return wg; + } + + public List<TPipelineWorkloadGroup> getWorkloadGroupByName(UserIdentity userIdentity, String wgFullName) Review Comment: 之前是唯一的,但是在resource tag的功能下,用户可以指定多个tag,同名的但是不同tag的wg,返回的就是多个 -- 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