yiguolei commented on code in PR #50563:
URL: https://github.com/apache/doris/pull/50563#discussion_r2072788533
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/AlterRoutineLoadStmt.java:
##########
@@ -257,19 +257,18 @@ private void checkJobProperties() throws UserException {
ConnectContext tmpCtx = new ConnectContext();
tmpCtx.setCurrentUserIdentity(ConnectContext.get().getCurrentUserIdentity());
tmpCtx.getSessionVariable().setWorkloadGroup(workloadGroup);
- long wgId;
try {
// NOTE(wb): why get 0th wg here;
// currently a routineload can only be executed in one
workload group;
// but the workload group thrift sent from FE to BE is a
list, this is for scalability.
- wgId = Env.getCurrentEnv().getWorkloadGroupMgr()
+ Env.getCurrentEnv().getWorkloadGroupMgr()
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]
