[
https://issues.apache.org/jira/browse/FLINK-31105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17690282#comment-17690282
]
Jing Ge commented on FLINK-31105:
---------------------------------
[~mapohl] It is more about the user behaviour consistency. Just like we use
semicolon in Java, Scala and Python don't. But all of them have their own
consistency (Scala case is not very clean, that's why there are lots of
discussion because users are confused). Back to SQL, even those commands are
not standard, the still belong to Flink (new) SQL. They should stick to the
standard, i.e. semicolon is required.
[~Sergey Nuyanzin] generally speaking, I am with you, it is a good idea to help
users with autocompletion. But not including semicolon because again we should
care about the behaviour consistency. If only some cases support the semicolon
autocompletion, most of others are not, users will be confused and start asking
more autocompletion for all other cases, like e.g. with (...), after ")" and
press TAB, the semicolon autocompletion will also be expected because of the
behaviour consistency. There will be more and more cases rising.
>From design and implementation's perspective, when more corner case
>requirements are rising, it will quickly become complicated if we don't want
>to use hard coding if...else and offer a general solution. A general solution
>might make wrong decision and add semicolon at the place where semicolon is
>not required or not required yet. Afaik, that is why most IDEs e.g. Intellij
>Idea offer excellent autocompletion but don't take care of corner case of
>semicolon autocompletion.
> SqlClient: Make HELP, EXIT and QUIT also work without a semicolon
> -----------------------------------------------------------------
>
> Key: FLINK-31105
> URL: https://issues.apache.org/jira/browse/FLINK-31105
> Project: Flink
> Issue Type: Improvement
> Components: Table SQL / Client
> Affects Versions: 1.17.0, 1.16.1
> Reporter: Matthias Pohl
> Priority: Minor
> Labels: starter
>
> The special commands {{HELP}}, {{EXIT}}, and {{QUIT}} are not really part of
> the SQL syntax but rather tool-specific commands. Having the requirement to
> have the semicolon follow-up on these commands might not be necessary.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
