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

    https://github.com/apache/spark/pull/22386#discussion_r216723249
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala ---
    @@ -1029,6 +1030,35 @@ class StreamSuite extends StreamTest {
             false))
       }
     
    +  test("is_continuous_processing property should be false for microbatch 
processing") {
    +    val input = MemoryStream[Int]
    +    val df = input.toDS()
    +      .map(i => 
TaskContext.get().getLocalProperty(ContinuousExecution.IS_CONTINUOUS_PROCESSING))
    +    testStream(df) (
    +      AddData(input, 1),
    +      CheckAnswer("false")
    +    )
    +  }
    +
    +  test("is_continuous_processing property should be true for continuous 
processing") {
    +    val input = ContinuousMemoryStream[Int]
    +    var x: String = ""
    --- End diff --
    
    Non used var?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to