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

    https://github.com/apache/spark/pull/21919#discussion_r208107058
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
 ---
    @@ -46,6 +46,9 @@ case class WriteToDataSourceV2(writer: DataSourceWriter, 
query: LogicalPlan) ext
      * The physical plan for writing data into data source v2.
      */
     case class WriteToDataSourceV2Exec(writer: DataSourceWriter, query: 
SparkPlan) extends SparkPlan {
    --- End diff --
    
    At the moment it is I think overkill. Maybe something to consider for 
future though as long as we wont find any use for those functionalities we will 
be adding. The one added now, may would have some use.


---

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

Reply via email to