turn off caching in the constructor

Author: marmbrus

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[_]) {

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.