Skip to content

Commit 1923598

Browse files
committed
[SPARK-32003] Minor tweak to comments.
1 parent a8e619c commit 1923598

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1934,8 +1934,8 @@ private[spark] class DAGScheduler(
19341934
* modify the scheduler's internal state. Use executorLost() to post a loss event from outside.
19351935
*
19361936
* We will also assume that we've lost all shuffle blocks associated with the executor if the
1937-
* executor serves its own blocks (i.e., we're not using external shuffle), or the Standalone
1938-
* worker (which serves the shuffle data) is lost.
1937+
* executor serves its own blocks (i.e., we're not using an external shuffle service), or the
1938+
* entire Standalone worker is lost.
19391939
*/
19401940
private[scheduler] def handleExecutorLost(
19411941
execId: String,
@@ -1958,9 +1958,8 @@ private[spark] class DAGScheduler(
19581958
* @param execId executor to be removed
19591959
* @param fileLost If true, indicates that we assume we've lost all shuffle blocks associated
19601960
* with the executor; this happens if the executor serves its own blocks (i.e., we're not
1961-
* using external shuffle), the Standalone worker (which serves the shuffle data) is lost,
1962-
* or a FetchFailed occurred (in which case we presume all shuffle data related to this
1963-
* executor to be lost).
1961+
* using an external shuffle service), the entire Standalone worker is lost, or a FetchFailed
1962+
* occurred (in which case we presume all shuffle data related to this executor to be lost).
19641963
* @param hostToUnregisterOutputs (optional) executor host if we're unregistering all the
19651964
* outputs on the host
19661965
* @param maybeEpoch (optional) the epoch during which the failure was caught (this prevents

0 commit comments

Comments
 (0)