Skip to content
Snippets Groups Projects
Commit a8a35b39 authored by Sandeep Singh's avatar Sandeep Singh Committed by Sean Owen
Browse files

[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.
parent 97da4103
No related branches found
No related tags found
No related merge requests found
...@@ -49,9 +49,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils { ...@@ -49,9 +49,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils {
protected override def beforeAll(): Unit = { protected override def beforeAll(): Unit = {
super.beforeAll() super.beforeAll()
sql("DROP TABLE IF EXISTS parquet_t0") (0 to 3).foreach { i =>
sql("DROP TABLE IF EXISTS parquet_t1") sql(s"DROP TABLE IF EXISTS parquet_t$i")
sql("DROP TABLE IF EXISTS parquet_t2") }
sql("DROP TABLE IF EXISTS t0") sql("DROP TABLE IF EXISTS t0")
spark.range(10).write.saveAsTable("parquet_t0") spark.range(10).write.saveAsTable("parquet_t0")
...@@ -87,10 +87,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils { ...@@ -87,10 +87,9 @@ class LogicalPlanToSQLSuite extends SQLBuilderTest with SQLTestUtils {
override protected def afterAll(): Unit = { override protected def afterAll(): Unit = {
try { try {
sql("DROP TABLE IF EXISTS parquet_t0") (0 to 3).foreach { i =>
sql("DROP TABLE IF EXISTS parquet_t1") sql(s"DROP TABLE IF EXISTS parquet_t$i")
sql("DROP TABLE IF EXISTS parquet_t2") }
sql("DROP TABLE IF EXISTS parquet_t3")
sql("DROP TABLE IF EXISTS t0") sql("DROP TABLE IF EXISTS t0")
} finally { } finally {
super.afterAll() super.afterAll()
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment