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

    https://github.com/apache/spark/pull/9988#discussion_r45938645
  
    --- Diff: 
streaming/src/test/scala/org/apache/spark/streaming/CheckpointSuite.scala ---
    @@ -33,17 +33,133 @@ import org.mockito.Mockito.mock
     import org.scalatest.concurrent.Eventually._
     import org.scalatest.time.SpanSugar._
     
    -import org.apache.spark.TestUtils
    +import org.apache.spark.{SparkConf, SparkContext, SparkFunSuite, TestUtils}
     import org.apache.spark.streaming.dstream.{DStream, FileInputDStream}
     import org.apache.spark.streaming.scheduler._
     import org.apache.spark.util.{MutableURLClassLoader, Clock, ManualClock, 
Utils}
     
     /**
    + * A trait of that can be mixed in to get methods for testing DStream 
operations under
    + * DStream checkpointing. Note that the implementations of this trait has 
to implement
    + * the `setupCheckpointOperation`
    + */
    +trait DStreamCheckpointTester { self: SparkFunSuite =>
    --- End diff --
    
    This is refactoring where I extract out the `testCheckpointedOperation` so 
that it can be used in other unit tests.


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to