address comments

DazhuangSu · DazhuangSu · commit 182449b76e67 · 2018-05-31T00:47:15.000+08:00
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
@@ -293,10 +293,10 @@ class AstBuilder(conf: SQLConf) extends SqlBaseBaseVisitor[AnyRef] with Logging
         case cmp @ BinaryComparison(UnresolvedAttribute(name :: Nil), constant: Literal) =>
           cmp
         case bc @ BinaryComparison(constant: Literal, _) =>
-          throw new ParseException("Literal " + constant
-            + " is supported only on the rigth-side.", ctx)
+          throw new ParseException(s"Literal $constant is supported only on the rigth-side.", ctx)
         case _ =>
-          throw new ParseException("Invalid partition filter specification", ctx)
+          throw new ParseException(
+            s"Invalid partition filter specification (${pVal.getText}).", ctx)
       }
     }
     parts

Original file line number	Diff line number	Diff line change
`@@ -293,10 +293,10 @@ class AstBuilder(conf: SQLConf) extends SqlBaseBaseVisitor[AnyRef] with Logging`
`293`	`293`	`case cmp @ BinaryComparison(UnresolvedAttribute(name :: Nil), constant: Literal) =>`
`294`	`294`	`cmp`
`295`	`295`	`case bc @ BinaryComparison(constant: Literal, _) =>`
`296`		`- throw new ParseException("Literal " + constant`
`297`		`- + " is supported only on the rigth-side.", ctx)`
	`296`	`+ throw new ParseException(s"Literal $constant is supported only on the rigth-side.", ctx)`
`298`	`297`	`case _ =>`
`299`		`- throw new ParseException("Invalid partition filter specification", ctx)`
	`298`	`+ throw new ParseException(`
	`299`	`+ s"Invalid partition filter specification (${pVal.getText}).", ctx)`
`300`	`300`	`}`
`301`	`301`	`}`
`302`	`302`	`parts`