turboFei commented on issue #26086: [SPARK-29302] Make the file name of a task 
for dynamic partition overwrite be unique
URL: https://github.com/apache/spark/pull/26086#issuecomment-547680198
 
 
   @viirya Thanks for your reply.
   
   > Not in all cases the written paths will be kept in allAbsPathFiles and so 
filesToMove. 
   
   Yes you are right,  So in this PR, I only fix the issue for dynamic 
partition overwrite, and there is no risk.
   
   > Actually for the cases filepath was put into addedAbsPathFiles, there is 
an UUID added, so the filename is unique already even without this patch.
   
   Yes, there is no risk for absPathFiles.
   But there is risk for dynamic partition overwrite.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to