samsetegne commented on a change in pull request #27567: [SPARK-30822][SQL]
Remove semicolon at the end of a sql query
URL: https://github.com/apache/spark/pull/27567#discussion_r388504762
##########
File path:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4
##########
@@ -64,7 +64,7 @@ grammar SqlBase;
}
singleStatement
- : statement EOF
+ : statement (';')*? EOF
Review comment:
@cloud-fan I would expect the most common scenario would be the user
unintentionally submitted an extra semicolon. Do you think the query should
fail in that case or is the intention so obvious that it's essentially
explicit? In cases where multiple statements are allowed i.e. spark-sql, would
the additional semicolons just equate to empty statements that are ignored or
would something else happen that could have an effect on performance?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
