Skip to content

Commit

Permalink
[SPARK-48768][PYTHON][CONNECT] Should not cache explain
Browse files Browse the repository at this point in the history
### 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 apache#47163 from zhengruifeng/should_not_cache_explain.

Authored-by: Ruifeng Zheng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
  • Loading branch information
zhengruifeng committed Jul 1, 2024
1 parent 703b076 commit 5c29d8d
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion python/pyspark/sql/connect/dataframe.py
Original file line number Diff line number Diff line change
Expand Up @@ -1975,7 +1975,6 @@ def _explain_string(
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:
Expand Down

0 comments on commit 5c29d8d

Please sign in to comment.