Fix parsing issues.

Author: concretevitamin

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/SqlParser.scala

@@ -179,9 +179,10 @@ class SqlParser extends StandardTokenParsers with PackratParsers {
   )
 
   protected lazy val setCommand: Parser[LogicalPlan] = {
+    // Comma needed for values such as "table1,table2".
     val keyVal = ident | numericLit | stringLit
-    (SET ~> keyVal <~ "=") ~ keyVal <~ opt(";") ^^ {
-      case key ~ value => SetCommand(key, value)
+    (SET ~> keyVal <~ "=") ~ rep1sep(keyVal, ",") <~ opt(";") ^^ {
+      case key ~ value => SetCommand(key, value.mkString(","))
     }
   }
 

sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala

@@ -30,9 +30,6 @@ private[sql] abstract class SparkStrategies extends QueryPlanner[SparkPlan] {
 
   object HashJoin extends Strategy with PredicateHelper {
 
-//    var broadCastTables =
-//    sqlConf.get("spark.sql.hints.broadcastTables", "").split(",")
-
     def apply(plan: LogicalPlan): Seq[SparkPlan] = plan match {
       // Find inner joins where at least some predicates can be evaluated by matching hash keys
       // using the HashFilteredJoin pattern.

sql/core/src/test/scala/org/apache/spark/sql/SQLConfSuite.scala

@@ -58,6 +58,14 @@ class SQLConfSuite extends QueryTest {
     assert(conf.get("mapred.reduce.tasks", "0") == "20")
     sql("set mapred.reduce.tasks = 40")
     assert(conf.get("mapred.reduce.tasks", "0") == "40")
+
+
+    val key = "spark.sql.key"
+    val vs = "val0,val_1,val2.3,my_table"
+    sql(s"set $key=$vs")
+    assert(conf.get(key, "0") == vs)
+
+    conf.clear()
   }
 
 }