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 <[email protected]>
Authored: Tue Nov 19 16:59:42 2013 -0800
Committer: Mark Hamstra <[email protected]>
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.

Reply via email to