Github user jose-torres commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20225#discussion_r161101942
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
 ---
    @@ -258,13 +276,9 @@ class ContinuousStressSuite extends 
ContinuousSuiteBase {
           AwaitEpoch(0),
           Execute(waitForRateSourceTriggers(_, 201)),
           IncrementEpoch(),
    -      Execute { query =>
    -        val data = query.sink.asInstanceOf[MemorySinkV2].allData
    -        val vals = data.map(_.getLong(0)).toSet
    -        assert(scala.Range(0, 25000).forall { i =>
    -          vals.contains(i)
    -        })
    -      })
    +      StopStream,
    --- End diff --
    
    This is the earlier comment about the overloaded failure mode this PR 
exposed.


---

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

Reply via email to