Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20386#discussion_r164648356
  
    --- Diff: 
sql/core/src/main/java/org/apache/spark/sql/sources/v2/writer/DataSourceV2Writer.java
 ---
    @@ -40,11 +40,13 @@
      *   1. Create a writer factory by {@link #createWriterFactory()}, 
serialize and send it to all the
      *      partitions of the input data(RDD).
      *   2. For each partition, create the data writer, and write the data of 
the partition with this
    - *      writer. If all the data are written successfully, call {@link 
DataWriter#commit()}. If
    - *      exception happens during the writing, call {@link 
DataWriter#abort()}.
    - *   3. If all writers are successfully committed, call {@link 
#commit(WriterCommitMessage[])}. If
    + *      writer. If all the data are written successfully, call {@link 
DataWriter#commit()}.
    --- End diff --
    
    If one data writer finishes successfully, the commit message will be sent 
back to the driver side and Spark will call #add.


---

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

Reply via email to