From a8a35b39b92fc9000eaac102c67c66be30b05e54 Mon Sep 17 00:00:00 2001
From: Sandeep Singh <sandeep@techaddict.me>
Date: Sat, 3 Sep 2016 15:35:19 +0100
Subject: [PATCH] [MINOR][SQL] Not dropping all necessary tables
## What changes were proposed in this pull request?
was not dropping table `parquet_t3`
## How was this patch tested?
tested `LogicalPlanToSQLSuite` locally
Author: Sandeep Singh <sandeep@techaddict.me>
Closes #13767 from techaddict/minor-8.
---
.../spark/sql/catalyst/LogicalPlanToSQLSuite.scala | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala b/sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala
index 3e0fdc1f8b..d80f894c22 100644
--- a/sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala
+++ b/sql/hive/src/test/scala/org/apache/spark/sql/catalyst/LogicalPlanToSQLSuite.scala
@@ -49,9 +49,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils {
protected override def beforeAll(): Unit = {
super.beforeAll()
- sql("DROP TABLE IF EXISTS parquet_t0")
- sql("DROP TABLE IF EXISTS parquet_t1")
- sql("DROP TABLE IF EXISTS parquet_t2")
+ (0 to 3).foreach { i =>
+ sql(s"DROP TABLE IF EXISTS parquet_t$i")
+ }
sql("DROP TABLE IF EXISTS t0")
spark.range(10).write.saveAsTable("parquet_t0")
@@ -87,10 +87,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils {
override protected def afterAll(): Unit = {
try {
- sql("DROP TABLE IF EXISTS parquet_t0")
- sql("DROP TABLE IF EXISTS parquet_t1")
- sql("DROP TABLE IF EXISTS parquet_t2")
- sql("DROP TABLE IF EXISTS parquet_t3")
+ (0 to 3).foreach { i =>
+ sql(s"DROP TABLE IF EXISTS parquet_t$i")
+ }
sql("DROP TABLE IF EXISTS t0")
} finally {
super.afterAll()
--
GitLab