morrySnow commented on code in PR #32379:
URL: https://github.com/apache/doris/pull/32379#discussion_r1528388827
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/CreateJobStmt.java:
##########
@@ -208,10 +210,24 @@ private void analyzerSqlStmt() throws UserException {
* parse execute sql from create job stmt
* Some stmt not implement toSql method,so we need to parse sql from
originStmt
*/
- private String parseExecuteSql(String sql) throws AnalysisException {
+ private static String parseExecuteSql(String sql, String jobName, String
comment) throws AnalysisException {
String lowerCaseSql = sql.toLowerCase();
- int executeSqlIndex = lowerCaseSql.indexOf(" do ");
- String executeSql = sql.substring(executeSqlIndex + 4).trim();
+ String lowerCaseJobName = jobName.toLowerCase();
Review Comment:
nit: maybe it is better to impl create job stmt in Nereids sometime
--
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
