[GitHub] [spark] steveloughran commented on pull request #21329: [SPARK-24277][SQL] Code clean up in SQL module: HadoopMapReduceCommitProtocol

[GitBox <gi...@apache.org>]

steveloughran commented on pull request #21329:
URL: https://github.com/apache/spark/pull/21329#issuecomment-715352031


   Very late commentary here: committers may expect a consistent jobID across job and task; reverting this was the right thing to do. Still got some race conditions in FileFormatWriter though, as >1 job can be created with same job ID


----------------------------------------------------------------
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:
users@infra.apache.org



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