wypoon commented on pull request #28848: URL: https://github.com/apache/spark/pull/28848#issuecomment-648339593
> sorry if I wasn't clear. I think this approach of having the fileLostEpoch is better so we avoid the locking in MapOutputTracker. Personally I wouldn't mind fileLost being renamed to something more obvious, like unregisterShuffleOutput . Thanks for the clarification and the suggestion. (In your earlier comment, the impression you gave me was "On the one hand, ...; on the other hand, ..." and it wasn't clear to me which side you came down on.) ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org