Skip to content

Commit 16a54ba

Browse files
author
Andrew Or
committed
Fix HiveQuerySuite?
Every time we called TestHive.reset() we created a new temp directory for derby, and then we would go ahead and override the old one in the same TestHiveContext. This fails tests that use multiple sessions for some reason. Setting the same confs in metadataHive whenever we call reset() seems unnecessary, so I removed it.
1 parent 98751cc commit 16a54ba

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -499,10 +499,7 @@ class TestHiveContext private[hive](
499499
// Lots of tests fail if we do not change the partition whitelist from the default.
500500
runSqlHive("set hive.metastore.partition.name.whitelist.pattern=.*")
501501

502-
TestHiveContext.hiveClientConfigurations(hiveconf, warehousePath, scratchDirPath)
503-
.foreach { case (k, v) => metadataHive.runSqlHive(s"SET $k=$v") }
504502
defaultOverrides()
505-
506503
sessionState.catalog.setCurrentDatabase("default")
507504
} catch {
508505
case e: Exception =>

0 commit comments

Comments
 (0)