Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/19150
  
    Looks `stop ensures correct shutdown` in `BlockGeneratorSuite` is dependent 
on interrupting - http://www.scalatest.org/release_notes/3.0.0:
    
    > If you were relying on the default behavior of interrupting a thread on 
the JVM in ScalaTest 2.2.x, you'll need to define an implicit val referring to 
a `ThreadSignaler`
    
    ScalaTest looks they changed the default for good reasons bug looks we 
should explicitly set `ThreadSignaler` to keep the previous behaviour more 
conservatively.
    
    ```scala
    implicit val defaultSignaler: Signaler = ThreadSignaler
    ```
    
    I just double checked this passes the pending tests.


---

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

Reply via email to