You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ma...@apache.org on 2013/12/06 20:50:04 UTC
[03/12] git commit: Fixed intended side-effects
Fixed intended side-effects
Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/982797dc
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/982797dc
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/982797dc
Branch: refs/heads/master
Commit: 982797dcbafa4c1149ad354b0c5a07e3f74fe005
Parents: 6f8359b
Author: Mark Hamstra <ma...@gmail.com>
Authored: Tue Nov 19 16:59:42 2013 -0800
Committer: Mark Hamstra <ma...@gmail.com>
Committed: Tue Dec 3 09:57:31 2013 -0800
----------------------------------------------------------------------
.../src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-spark/blob/982797dc/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala b/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
index ad436f8..bf5827d 100644
--- a/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
+++ b/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala
@@ -568,8 +568,8 @@ class DAGScheduler(
val activeInGroup = activeJobs.filter(groupId == _.properties.get(SparkContext.SPARK_JOB_GROUP_ID))
val jobIds = activeInGroup.map(_.jobId)
jobIds.foreach { handleJobCancellation }
- activeJobs -- activeInGroup
- idToActiveJob -- jobIds
+ activeJobs --= activeInGroup
+ idToActiveJob --= jobIds
case AllJobsCancelled =>
// Cancel all running jobs.