yiguolei commented on code in PR #48187:
URL: https://github.com/apache/doris/pull/48187#discussion_r1969705644
##########
fe/fe-core/src/main/java/org/apache/doris/httpv2/rest/LoadAction.java:
##########
@@ -439,18 +440,24 @@ private TNetworkAddress
selectLocalRedirectBackend(boolean groupCommit, HttpServ
throws LoadException {
Backend backend = null;
BeSelectionPolicy policy = null;
- String qualifiedUser = ConnectContext.get().getQualifiedUser();
- Set<Tag> userTags =
Env.getCurrentEnv().getAuth().getResourceTags(qualifiedUser);
- policy = new BeSelectionPolicy.Builder()
- .addTags(userTags)
- .setEnableRoundRobin(true)
- .needLoadAvailable().build();
- policy.nextRoundRobinIndex = getLastSelectedBackendIndexAndUpdate();
List<Long> backendIds;
if (groupCommit) {
+ String qualifiedUser = ConnectContext.get().getQualifiedUser();
+ Set<Tag> userTags =
Env.getCurrentEnv().getAuth().getComputeGroupTags(qualifiedUser);
Review Comment:
如果这里返回的明确知道是compute group tags，为啥不直接返回compute group 对象？ 这样447
行的代码里，就不需要调用add tags了
--
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
