cloud-fan commented on a change in pull request #35569:
URL: https://github.com/apache/spark/pull/35569#discussion_r811107313



##########
File path: 
core/src/main/scala/org/apache/spark/internal/io/HadoopMapReduceCommitProtocol.scala
##########
@@ -157,6 +159,7 @@ class HadoopMapReduceCommitProtocol(
     // Include a UUID here to prevent file collisions for one task writing to 
different dirs.
     // In principle we could include hash(absoluteDir) instead but this is 
simpler.
     val tmpOutputPath = new Path(stagingDir, UUID.randomUUID().toString() + 
"-" + filename).toString
+    stagingDirExists = true

Review comment:
       This is called at the executor side. How does it work with the checking 
code in `commitJob` and `abortJob`?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to