yiguolei commented on code in PR #49884:
URL: https://github.com/apache/doris/pull/49884#discussion_r2034619945
##########
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:
这个接口返回值为啥是一个Thrfit？ 另外，我们为啥会是一个list，难道还不唯一？
--
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
