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

   ### What changes were proposed in this pull request?
   
   Follow-up to #57582 (SPARK-58389), which forwarded user options to the table 
read path
   but not the `RelationCatalog` single-RPC path. This PR adds
   `RelationCatalog.loadRelation(Identifier, CaseInsensitiveStringMap)` 
(default ignores the
   options and delegates to `loadRelation(Identifier)`, so existing connectors 
are
   unaffected) and forwards the user options to it from `RelationResolution`.
   
   ### Why are the changes needed?
   
   For a `RelationCatalog` (a connector exposing both tables and views), 
`loadRelation` is
   the primary plain-read path, and it currently drops the user options. This 
is the
   single-RPC counterpart of #57582 and lets such connectors see the read 
options when
   producing the relation.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. New connector-API entry point with a behavior-preserving default; not 
yet released.
   
   ### How was this patch tested?
   
   New test in `DataSourceV2OptionSuite` asserting a `spark.read.option(...)` 
value reaches
   `loadRelation`. Regression: `DataSourceV2OptionSuite`, 
`DataSourceV2MetadataViewSuite`,
   `MetricViewV2CatalogSuite`, `SupportsCatalogOptionsSuite` (76/76); 
`DataSourceV2SQLSuite`
   (V1/V2 filter) and the v2 view command suites (572/572).
   
   ### 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