Skip to content

Commit 9ebe380

Browse files
ash211pwoody
authored andcommitted
SPARK-18113 Authorize duplicate requests to commit (apache#79)
Commonly encountered because of dropped responses from the driver back to an executor authorizing a commit.
1 parent f9fc3a6 commit 9ebe380

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,11 @@ private[spark] class OutputCommitCoordinator(conf: SparkConf, isDriver: Boolean)
164164
s"partition=$partition")
165165
authorizedCommitters(partition) = attemptNumber
166166
true
167+
case existingCommitter if existingCommitter == attemptNumber =>
168+
logWarning(s"Authorizing duplicate request to commit for " +
169+
s"attemptNumber=$attemptNumber to commit for stage=$stage, partition=$partition; " +
170+
s"existingCommitter = $existingCommitter. This can indicate dropped network traffic.")
171+
true
167172
case existingCommitter =>
168173
logDebug(s"Denying attemptNumber=$attemptNumber to commit for stage=$stage, " +
169174
s"partition=$partition; existingCommitter = $existingCommitter")

0 commit comments

Comments
 (0)