yiguolei commented on code in PR #50563:
URL: https://github.com/apache/doris/pull/50563#discussion_r2072789053
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/CreateRoutineLoadStmt.java:
##########
@@ -589,13 +589,14 @@ private void checkJobProperties() throws UserException {
// 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.
- this.workloadGroupId =
Env.getCurrentEnv().getWorkloadGroupMgr()
+ Env.getCurrentEnv().getWorkloadGroupMgr()
Review Comment:
不要依赖这种异常来检查properties，比如这个wg 不存在，你这个异常抛出的是index out of bound
exception，这种用户不理解。
--
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]
