uros-b opened a new pull request, #57846:
URL: https://github.com/apache/spark/pull/57846

   ### What changes were proposed in this pull request?
   
   Replaces a stale sentence in the caching section of 
`docs/sql-performance-tuning.md`. The old text told users to call 
`spark.catalog.listCachedTables()`; the new text points to 
`spark.catalog.isCached`, the `Dataset.storageLevel` property, and the web UI 
Storage tab.
   
   ### Why are the changes needed?
   
   `spark.catalog.listCachedTables()` no longer exists. It was removed by the 
SPARK-56221 followup (`a3930d31665`) along with the `SHOW CACHED TABLES` 
command. That commit edited this sentence to drop the SQL command but left the 
`listCachedTables()` call in place, so the guide currently points at a method 
that cannot be called. This is the only remaining reference to the removed API 
in the repository.
   
   Each replacement is a documented API: `Catalog.isCached` 
(`sql/api/.../catalog/Catalog.scala`), `Dataset.storageLevel`, whose 
implementation returns `StorageLevel.NONE` when the data is not cached, and the 
Storage tab, which `docs/web-ui.md` describes as showing "the storage levels, 
sizes and partitions of all RDDs" and notes does not list a relation "before 
they are materialized". The Storage tab also covers data cached via 
`Dataset.cache()` without a name, which is what the removed API could not 
report.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No, other than the corrected documentation.
   
   ### How was this patch tested?
   
   Documentation only. The `#storage-tab` anchor matches the `## Storage Tab` 
heading in `docs/web-ui.md`, and the link style matches the existing `[SQL 
UI](web-ui.html#sql-tab)` reference in this same file.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Opus 4.8)
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to