mihailom-db commented on code in PR #48951:
URL: https://github.com/apache/spark/pull/48951#discussion_r1893566514
##########
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala:
##########
@@ -106,14 +106,14 @@ class SparkSqlAstBuilder extends AstBuilder {
SetCommand(Some(keyStr -> None))
}
} else {
- remainder(ctx.SET.getSymbol).trim match {
+ remainder(ctx.SET.getSymbol).trim.replaceAll(";+$", "") match {
Review Comment:
Well, `.trimRight(";")` can only be done on `UTF8String`, but here we have
java `String`. In java we do not have formal trimRight or trimLeft, but only
trim for spaces. This is a regex replace to exchange all occurrences of `;` at
the end ($) of the String.
--
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]
