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

    https://github.com/apache/spark/pull/18975#discussion_r137680153
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
 ---
    @@ -360,6 +360,31 @@ case class InsertIntoTable(
     }
     
     /**
    + * Insert query result into a directory.
    + *
    + * @param isLocal Indicates whether the specified directory is local 
directory
    + * @param storage Info about output file, row and what serialization format
    + * @param provider Specifies what data source to use; only used for data 
source file.
    + * @param child The query to be executed
    + * @param overwrite If true, the existing directory will be overwritten
    + *
    + * Note that this plan is unresolved and has to be replaced by the 
concrete implementations
    + * 
sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scaladuring 
analysis.
    --- End diff --
    
    Could you fix it?


---

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

Reply via email to