Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22337#discussion_r216120459
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/streaming/state/StateStoreRDDSuite.scala
 ---
    @@ -49,8 +49,11 @@ class StateStoreRDDSuite extends SparkFunSuite with 
BeforeAndAfter with BeforeAn
       }
     
       override def afterAll(): Unit = {
    -    super.afterAll()
    -    Utils.deleteRecursively(new File(tempDir))
    +    try {
    +      Utils.deleteRecursively(new File(tempDir))
    +    } finally {
    +      super.afterAll()
    --- End diff --
    
    Ur, are you sure? `StreamingAggregationSuite`, `StreamTest`, 
`StreamingAggregationSuite` and `StreamingDeduplicationSuite` also have reverse 
order for `StateStore.stop()`.


---

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

Reply via email to