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

    https://github.com/apache/spark/pull/19893#discussion_r159954825
  
    --- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
    @@ -27,19 +27,55 @@ import org.apache.spark.util.AccumulatorContext
     
     /**
      * Base abstract class for all unit tests in Spark for handling common 
functionality.
    + *
    + * Thread audit happens normally here automatically when a new test suite 
created.
    + * The only prerequisite for that is that the test class must extend 
[[SparkFunSuite]].
    + *
    + * There are some test suites which are doing initialization before 
[[SparkFunSuite#beforeAll]]
    --- End diff --
    
    Better:
    
    "
    It is possible to override the default thread audit behavior by setting 
`enableAutoThreadAudit` to false and manually calling the audit methods, if 
desired. For example:
    
    // Code
    "


---

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

Reply via email to