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