Skip to content

Commit

Permalink
[SPARK-32003] One more comment tweak.
Browse files Browse the repository at this point in the history
  • Loading branch information
wypoon committed Jul 13, 2020
1 parent 4301291 commit 56f3ae5
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -189,9 +189,9 @@ private[spark] class DAGScheduler(
* Tracks the latest epoch of a fully processed error where shuffle files have been lost from
* the given executor.
*
* This is closely related to executorFailureEpoch.
* They only differ for the executor when there is a Standalone worker or an external shuffle
* service serving shuffle files. In that case, when an executor is lost, we do not update
* This is closely related to executorFailureEpoch. They only differ for the executor when
* there is an external shuffle service serving shuffle files and we haven't been notified that
* the entire worker has been lost. In that case, when an executor is lost, we do not update
* the shuffleFileLostEpoch; we wait for a fetch failure. This way, if only the executor
* fails, we do not unregister the shuffle data as it can still be served; but if there is
* a failure in the shuffle service (resulting in fetch failure), we unregister the shuffle
Expand Down

0 comments on commit 56f3ae5

Please sign in to comment.