Skip to content

Commit

Permalink
Fix HiveQuerySuite?
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Andrew Or committed Mar 22, 2016
1 parent 98751cc commit 16a54ba
Showing 1 changed file with 0 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -499,10 +499,7 @@ class TestHiveContext private[hive](
// Lots of tests fail if we do not change the partition whitelist from the default.
runSqlHive("set hive.metastore.partition.name.whitelist.pattern=.*")

TestHiveContext.hiveClientConfigurations(hiveconf, warehousePath, scratchDirPath)
.foreach { case (k, v) => metadataHive.runSqlHive(s"SET $k=$v") }
defaultOverrides()

sessionState.catalog.setCurrentDatabase("default")
} catch {
case e: Exception =>
Expand Down

0 comments on commit 16a54ba

Please sign in to comment.