From 16a54bad76a8297f15417ba931d20c4d86092c84 Mon Sep 17 00:00:00 2001 From: Andrew Or Date: Tue, 22 Mar 2016 14:41:03 -0700 Subject: [PATCH] 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. --- .../main/scala/org/apache/spark/sql/hive/test/TestHive.scala | 3 --- 1 file changed, 3 deletions(-) diff --git a/sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala b/sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala index 1b352040f15d1..11559030374b9 100644 --- a/sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala +++ b/sql/hive/src/main/scala/org/apache/spark/sql/hive/test/TestHive.scala @@ -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 =>