Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/20133#discussion_r159170195 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/ParserUtils.scala --- @@ -39,6 +41,17 @@ object ParserUtils { throw new ParseException(s"Operation not allowed: $message", ctx) } + def duplicateClausesNotAllowed(message: String, ctx: ParserRuleContext): Nothing = { + throw new ParseException(s"Found duplicate clauses: $message", ctx) --- End diff -- Sounds good to me!
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org