Cheng's comments.

Author: yhuai

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

@@ -146,13 +146,9 @@ class SqlParser extends AbstractSparkSQLParser {
       }
 
   protected lazy val insert: Parser[LogicalPlan] =
-    ( INSERT ~> OVERWRITE ~> TABLE ~> relation ~ select ^^ {
-        case r ~ s => InsertIntoTable(r, Map.empty[String, Option[String]], s, true)
-      }
-    | INSERT ~> INTO ~> TABLE ~> relation ~ select ^^ {
-      case r ~ s => InsertIntoTable(r, Map.empty[String, Option[String]], s, false)
-      }
-    )
+    INSERT ~> (OVERWRITE ^^^ true | INTO ^^^ false) ~ (TABLE ~> relation) ~ select ^^ {
+      case o ~ r ~ s => InsertIntoTable(r, Map.empty[String, Option[String]], s, o)
+    }
 
   protected lazy val projection: Parser[Expression] =
     expression ~ (AS.? ~> ident.?) ^^ {

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

@@ -318,13 +318,13 @@ private[sql] abstract class SparkStrategies extends QueryPlanner[SparkPlan] {
         ExecutedCommand(
           CreateTempTableUsing(
             tableName, userSpecifiedSchema, provider, opts, allowExisting)) :: Nil
-      case CreateTableUsing(_, _, _, false, _, _) =>
+      case c: CreateTableUsing if !c.temporary =>
         sys.error("Tables created with SQLContext must be TEMPORARY. Use a HiveContext instead.")
 
       case CreateTableUsingAsSelect(tableName, provider, true, options, allowExisting, query) =>
         ExecutedCommand(
           CreateTempTableUsingAsSelect(tableName, provider, options, allowExisting, query)) :: Nil
-      case CreateTableUsingAsSelect(_, _, false, _, _, _) =>
+      case c: CreateTableUsingAsSelect if !c.temporary =>
         sys.error("Tables created with SQLContext must be TEMPORARY. Use a HiveContext instead.")
 
       case _ => Nil