Github user markhamstra commented on a diff in the pull request:

    https://github.com/apache/spark/pull/186#discussion_r11865385
  
    --- Diff: 
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
    @@ -18,28 +18,30 @@
     package org.apache.spark.scheduler
     
     import scala.Tuple2
    -import scala.collection.mutable.{HashSet, HashMap, Map}
    +import scala.collection.mutable.{HashMap, Map, HashSet}
     
    +import akka.actor._
    +import akka.testkit.{TestKit, ImplicitSender, TestActorRef}
     import org.scalatest.{BeforeAndAfter, FunSuite}
    +import org.scalatest.matchers.MustMatchers
     
     import org.apache.spark._
     import org.apache.spark.rdd.RDD
     import org.apache.spark.scheduler.SchedulingMode.SchedulingMode
     import org.apache.spark.storage.{BlockId, BlockManagerId, 
BlockManagerMaster}
    +import scala.collection.mutable
    +
    +class BuggyDAGEventProcessActor extends Actor {
    +  val state = 0
    +  def receive = {
    +    case _ => throw new SparkException("error")
    +  }
    +}
    +
    +class DAGSchedulerSuite
    +  extends TestKit(ActorSystem("DAGSchedulerSuite")) with FunSuite with 
MustMatchers
    --- End diff --
    
    MustMatchers isn't needed -- remove from imports, too


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to