Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22337#discussion_r215853929
--- 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 --
Could you check the history of this too? Is there a reason for this
`StateStoreRDDSuite` to put `deleteRecursively` at the end?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
