Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19155#discussion_r137638893
  
    --- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala 
---
    @@ -1603,7 +1617,7 @@ class SQLQuerySuite extends QueryTest with 
SQLTestUtils with TestHiveSingleton {
         val fs = path.getFileSystem(sparkContext.hadoopConfiguration)
         // The partition dir is not empty.
         assert(fs.listStatus(new Path(path, "part=1")).nonEmpty)
    -
    +    sql("DROP TABLE test_table")
    --- End diff --
    
    We don't need to do this. We will `DROP DATABASE test_database CASCADE` at 
[here](https://github.com/apache/spark/pull/19155/files#diff-1ea02a6fab84e938582f7f87cc4d9ea1R1622).


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to