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

    https://github.com/apache/spark/pull/22432#discussion_r217902092
  
    --- Diff: 
core/src/test/scala/org/apache/spark/util/collection/ExternalAppendOnlyMapSuite.scala
 ---
    @@ -457,7 +458,7 @@ class ExternalAppendOnlyMapSuite extends SparkFunSuite
         // 
https://github.com/scala/scala/blob/2.13.x/test/junit/scala/tools/testing/AssertUtil.scala
         // (lines 69-89)
         // assert(map.currentMap == null)
    -    eventually {
    +    eventually(timeout(5 seconds), interval(200 milliseconds)) {
    --- End diff --
    
    @dongjoon-hyun, thanks for cleaning up my mess! :sunglasses: 


---

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

Reply via email to