CalvinKirs commented on code in PR #32379:
URL: https://github.com/apache/doris/pull/32379#discussion_r1528403678
##########
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:
yup. I'll start a thread later, but I think this might be in 2.2.x or 3.0
--
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]
