dongjoon-hyun commented on a change in pull request #25427:
[SPARK-28705][Test]drop table after being used
URL: https://github.com/apache/spark/pull/25427#discussion_r313210333
##########
File path:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisExternalCatalogSuite.scala
##########
@@ -57,6 +57,7 @@ class AnalysisExternalCatalogSuite extends AnalysisTest with
Matchers {
val plan = Project(Seq(func), testRelation)
analyzer.execute(plan)
verifyZeroInteractions(catalog)
+ catalog.dropTable("default", "t1", true, false)
Review comment:
@sandeep-katta . This is a `inMemoryCatalog` created in this test case. The
catalog will not affect the others.
Do you need to remove the location used for `default.t1` instead of the
table itself?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
