Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/5029#discussion_r26536957
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertIntoHiveTableSuite.scala
---
@@ -136,6 +135,7 @@ class InsertIntoHiveTableSuite extends QueryTest with
BeforeAndAfter {
assert(listFolders(tmpDir,List()).sortBy(_.toString()) ==
expected.sortBy(_.toString))
sql("DROP TABLE table_with_partition")
sql("DROP TABLE tmp_table")
+ Utils.deleteRecursively(tmpDir)
--- End diff --
I was going to suggest a "TempDirectory" trait or something to manage a
temp dir per test. If scalatest runs a single JVM for all tests, then it might
be worth it, but it would be a very noisy change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
