httfighter commented on a change in pull request #24440: [SPARK-27545] [SQL]
Uncache table needs to delete the temporary view …
URL: https://github.com/apache/spark/pull/24440#discussion_r278377755
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala
##########
@@ -442,7 +442,11 @@ class CatalogImpl(sparkSession: SparkSession) extends
Catalog {
override def uncacheTable(tableName: String): Unit = {
Review comment:
@viirya Yes. There is indeed this problem. However, there is still a problem
of accidental deletion when removing the view at `UncacheTableCommand`. Because
when executing `uncache table`, I can't judge whether the temporary view is
created by the user or created when the `cache table` is executed.
I want to ask, why create a view when executing `cache table as select` .
What does this view do? Because there is no correspondence between `view `and
`plan`.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
