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

    https://github.com/apache/spark/pull/19893#discussion_r159527436
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/SessionStateSuite.scala ---
    @@ -39,6 +41,7 @@ class SessionStateSuite extends SparkFunSuite
       protected var activeSession: SparkSession = _
     
       override def beforeAll(): Unit = {
    +    doThreadPreAudit()
    --- End diff --
    
    Isn't the problem here that this is not calling `super.beforeAll()`? If you 
do that, you don't need to override `doThreadAuditInSparkFunSuite` nor call 
`doThreadPostAudit` below.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to