Github user viirya commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18975#discussion_r137918839
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
 ---
    @@ -192,12 +245,23 @@ class AstBuilder(conf: SQLConf) extends 
SqlBaseBaseVisitor[AnyRef] with Logging
             "partitions with value: " + 
dynamicPartitionKeys.keys.mkString("[", ",", "]"), ctx)
         }
     
    -    InsertIntoTable(
    -      UnresolvedRelation(tableIdent),
    -      partitionKeys,
    -      query,
    -      ctx.OVERWRITE != null,
    -      ctx.EXISTS != null)
    +    (tableIdent, partitionKeys, ctx.OVERWRITE() != null, ctx.EXISTS() != 
null)
    +  }
    +
    +  /**
    +   * Write to a file, returning a [[InsertIntoDir]] logical plan.
    --- End diff --
    
    Write to a dir?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to