[GitHub] [spark] ueshin commented on a diff in pull request #40015: [SPARK-42437][PYTHON][CONNECT] PySpark catalog.cacheTable will allow to specify storage level

2023-04-11 Thread via GitHub
ueshin commented on code in PR #40015: URL: https://github.com/apache/spark/pull/40015#discussion_r1163280656 ## python/pyspark/sql/catalog.py: ## @@ -917,25 +919,34 @@ def isCached(self, tableName: str) -> bool: """ return self._jcatalog.isCached(tableName)

[GitHub] [spark] ueshin commented on a diff in pull request #40015: [SPARK-42437][PYTHON][CONNECT] PySpark catalog.cacheTable will allow to specify storage level

2023-04-07 Thread via GitHub
ueshin commented on code in PR #40015: URL: https://github.com/apache/spark/pull/40015#discussion_r1160908837 ## python/pyspark/sql/connect/plan.py: ## @@ -1830,14 +1831,24 @@ def plan(self, session: "SparkConnectClient") -> proto.Relation: class CacheTable(LogicalPlan): -