cloud-fan commented on code in PR #49835:
URL: https://github.com/apache/spark/pull/49835#discussion_r1946030146
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AbstractSqlParser.scala:
##########
@@ -72,9 +72,9 @@ abstract class AbstractSqlParser extends AbstractParser with
ParserInterface {
/** Creates LogicalPlan for a given SQL string of query. */
override def parseQuery(sqlText: String): LogicalPlan =
parse(sqlText) { parser =>
- val ctx = parser.query()
+ val ctx = parser.singleQuery()
Review Comment:
This is a bug fix but also a breaking change, as some SQL may fail to be
parsed now. Shall we add a legacy config to restore to the old buggy behavior?
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
