turn off caching in the constructor

apache · Aug 23, 2014 · 5c7848d · 5c7848d
1 parent 8074a80
commit 5c7848d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 13 deletions.
diff --git a/sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala b/sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala
@@ -29,15 +29,8 @@ import org.apache.spark.sql.hive.test.TestHive
 import org.apache.spark.sql.hive.test.TestHive._
 
 class StatisticsSuite extends QueryTest with BeforeAndAfterAll {
-
-  override def beforeAll() = {
-    // HACK: Cached tables do not currently preserve statistics...
-    TestHive.cacheTables = false
-  }
-
-  override def afterAll() = {
-    TestHive.cacheTables = true
-  }
+  TestHive.reset()
+  TestHive.cacheTables = false
 
   test("parse analyze commands") {
     def assertAnalyzeCommand(analyzeCommand: String, c: Class[_]) {

diff --git a/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/PruningSuite.scala b/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/PruningSuite.scala
@@ -30,10 +30,7 @@ import scala.collection.JavaConversions._
 class PruningSuite extends HiveComparisonTest with BeforeAndAfter {
   // MINOR HACK: You must run a query before calling reset the first time.
   TestHive.sql("SHOW TABLES")
-
-  override def beforeAll() = {
-    TestHive.cacheTables = false
-  }
+  TestHive.cacheTables = false
 
   // Column/partition pruning is not implemented for `InMemoryColumnarTableScan` yet, need to reset
   // the environment to ensure all referenced tables in this suites are not cached in-memory.