Skip to content

Commit 3a6511f

Browse files
Remove the temp dir after text exit
1 parent b9df8af commit 3a6511f

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

sql/hive/src/main/scala/org/apache/spark/sql/hive/TestHive.scala

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import org.apache.hadoop.hive.serde2.`lazy`.LazySimpleSerDe
3131
import org.apache.hadoop.hive.serde2.avro.AvroSerDe
3232

3333
import org.apache.spark.{SparkConf, SparkContext}
34+
import org.apache.spark.util.Utils
3435
import org.apache.spark.sql.catalyst.analysis._
3536
import org.apache.spark.sql.catalyst.plans.logical.{CacheTableCommand, LogicalPlan, NativeCommand}
3637
import org.apache.spark.sql.catalyst.util._
@@ -71,11 +72,14 @@ class TestHiveContext(sc: SparkContext) extends HiveContext(sc) {
7172
setConf("javax.jdo.option.ConnectionURL",
7273
s"jdbc:derby:;databaseName=$metastorePath;create=true")
7374
setConf("hive.metastore.warehouse.dir", warehousePath)
75+
Utils.registerShutdownDeleteDir(new File(warehousePath))
76+
Utils.registerShutdownDeleteDir(new File(metastorePath))
7477
}
7578

7679
val testTempDir = File.createTempFile("testTempFiles", "spark.hive.tmp")
7780
testTempDir.delete()
7881
testTempDir.mkdir()
82+
Utils.registerShutdownDeleteDir(testTempDir)
7983

8084
// For some hive test case which contain ${system:test.tmp.dir}
8185
System.setProperty("test.tmp.dir", testTempDir.getCanonicalPath)
@@ -121,8 +125,7 @@ class TestHiveContext(sc: SparkContext) extends HiveContext(sc) {
121125
val hiveFilesTemp = File.createTempFile("catalystHiveFiles", "")
122126
hiveFilesTemp.delete()
123127
hiveFilesTemp.mkdir()
124-
hiveFilesTemp.deleteOnExit()
125-
128+
Utils.registerShutdownDeleteDir(hiveFilesTemp)
126129

127130
val inRepoTests = if (System.getProperty("user.dir").endsWith("sql" + File.separator + "hive")) {
128131
new File("src" + File.separator + "test" + File.separator + "resources" + File.separator)

0 commit comments

Comments
 (0)