gengliangwang opened a new pull request, #55616:
URL: https://github.com/apache/spark/pull/55616

   ### What changes were proposed in this pull request?
   
   Rename the unified listing method on `TableViewCatalog` from 
`listRelationSummaries` to `listTableAndViewSummaries`. Updates the declaration 
in `TableViewCatalog.java` and the two existing references introduced by 
[SPARK-56655](https://issues.apache.org/jira/browse/SPARK-56655):
   
   - `sql/catalyst/.../TableViewCatalog.java` -- method declaration + javadoc
   - `sql/core/.../ShowTablesExec.scala` -- call site + scaladoc reference
   - `sql/core/.../DataSourceV2MetadataViewSuite.scala` -- comment reference
   
   ### Why are the changes needed?
   
   The new name explicitly states what the method returns (tables and views) 
and avoids overloading Spark's existing "Relation" terminology (`BaseRelation`, 
`LogicalRelation`, etc.), matching the rename rationale that already changed 
`RelationCatalog` -> `TableViewCatalog` in SPARK-56655. The API is still 
`@Evolving` and unreleased, so this is a pre-release rename with no deprecation 
cycle needed.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes for **connector developers** overriding 
`TableViewCatalog#listRelationSummaries`: they need to update the override name 
to `listTableAndViewSummaries`. The API was added in the unreleased 
SPARK-56655, so this is pre-release breakage on an `@Evolving` interface.
   
   No end-user behavior change.
   
   ### How was this patch tested?
   
   Pure rename; existing tests in `DataSourceV2MetadataViewSuite` exercise the 
renamed method via `SHOW TABLES` on a `TableViewCatalog`.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: Claude (Anthropic), claude-opus-4


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