You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/02/11 11:47:37 UTC
[GitHub] [spark] cloud-fan commented on a change in pull request #27090:
[SPARK-27545][SQL][DOC] Update the Documentation for CACHE TABLE and
UNCACHE TABLE
cloud-fan commented on a change in pull request #27090: [SPARK-27545][SQL][DOC] Update the Documentation for CACHE TABLE and UNCACHE TABLE
URL: https://github.com/apache/spark/pull/27090#discussion_r377586700
##########
File path: docs/sql-ref-syntax-aux-cache-uncache-table.md
##########
@@ -21,11 +21,14 @@ license: |
### Description
`UNCACHE TABLE` removes the entries and associated data from the in-memory and/or on-disk cache for a given table or view. The
-underlying entries should already have been brought to cache by previous `CACHE TABLE` operation. `UNCACHE TABLE` on a non-existent table throws Exception if `IF EXISTS` is not specified.
+underlying entries should already have been brought to cache by previous `CACHE TABLE` operation. `UNCACHE TABLE` does not remove
+a temporary view. `UNCACHE TABLE` on a non-existent table throws an exception if `IF EXISTS` is not specified.
Review comment:
Do we need to highlight `UNCACHE TABLE does not remove a temporary view.`? It's intuitive that UNCACHE won't drop temp view.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org