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

    https://github.com/apache/spark/pull/21606#discussion_r197811782
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
 ---
    @@ -125,12 +124,12 @@ object DataWritingSparkTask extends Logging {
             val coordinator = SparkEnv.get.outputCommitCoordinator
             val commitAuthorized = coordinator.canCommit(stageId, 
stageAttempt, partId, attemptId)
    --- End diff --
    
    a note for the followup: since we decided to use `taskId` as a unique 
identifier for write tasks, the output coordinator can also use `taskId` 
instead of stage and task attempts.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to