This is an automated email from the ASF dual-hosted git repository.
ruifengz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 5c29d8d505a9 [SPARK-48768][PYTHON][CONNECT] Should not cache `explain`
5c29d8d505a9 is described below
commit 5c29d8d505a9167099c7113af58dca8fe09d2323
Author: Ruifeng Zheng <ruife...@apache.org>
AuthorDate: Tue Jul 2 07:37:00 2024 +0800
[SPARK-48768][PYTHON][CONNECT] Should not cache `explain`
### What changes were proposed in this pull request?
Should not cache `explain`
### Why are the changes needed?
the plans can be affected by `dataframe.cache`
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
CI
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #47163 from zhengruifeng/should_not_cache_explain.
Authored-by: Ruifeng Zheng <ruife...@apache.org>
Signed-off-by: Ruifeng Zheng <ruife...@apache.org>
---
python/pyspark/sql/connect/dataframe.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/python/pyspark/sql/connect/dataframe.py
b/python/pyspark/sql/connect/dataframe.py
index 1aa8fc00cfcc..46698c2530ea 100644
--- a/python/pyspark/sql/connect/dataframe.py
+++ b/python/pyspark/sql/connect/dataframe.py
@@ -1975,7 +1975,6 @@ class DataFrame(ParentDataFrame):
query = self._plan.to_proto(self._session.client)
return self._session.client.explain_string(query, explain_mode)
- @functools.cache
def explain(
self, extended: Optional[Union[bool, str]] = None, mode: Optional[str]
= None
) -> None:
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
