hackergin commented on code in PR #25988:
URL: https://github.com/apache/flink/pull/25988#discussion_r1921709924
##########
flink-table/flink-sql-gateway/src/main/java/org/apache/flink/table/gateway/service/application/ScriptExecutor.java:
##########
@@ -194,6 +194,7 @@ public boolean hasNext() {
return true;
}
}
+ position = script.length();
Review Comment:
The `position` keeps track of the current location in the SQL text being
parsed. Previously, the logic only updated the position when a semicolon was
encountered. However, if there is no semicolon at the end of the text—meaning
we’ve finished processing all the text without updating the `position`—the next
time we attempt to split, it will start from the previous `position` again.
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
