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

Reply via email to