Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22263#discussion_r213904343
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -162,7 +162,8 @@ statement
tableIdentifier partitionSpec? describeColName?
#describeTable
| REFRESH TABLE tableIdentifier
#refreshTable
| REFRESH (STRING | .*?)
#refreshResource
- | CACHE LAZY? TABLE tableIdentifier (AS? query)?
#cacheTable
+ | CACHE LAZY? storageLevel=identifier? TABLE
+ tableIdentifier (AS? query)?
#cacheTable
--- End diff --
As another syntax option, how about this?
```
CACHE TABLE testTable OPTIONS ('storageLevel'='disk_only')
UNCACHE TABLE testTable OPTION ('cascade'='true', 'blocking'='true')
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
