HeartSaVioR opened a new pull request, #48605: URL: https://github.com/apache/spark/pull/48605
### What changes were proposed in this pull request? This PR proposes to introduce a new pattern object to extract BaseRelation and CatalogTable from LogicalRelation, and update existing pattern matches with LogicalRelation with the new pattern. ### Why are the changes needed? I found it very difficult to add a param into LogicalRelation, because a bunch of places use pattern matching with LogicalRelation, and all of them are using default pattern with specifying all params (despite wildcards). I looked into the patterns and realized most of the patterns just need relation and catalog table inside the LogicalRelation. output and isStreaming are used infrequently. That said, introducing a new pattern object extracting BaseRelation and CatalogTable from LogicalRelation would cover most of the patterns we have for LogicalRelation, and this would help a lot to introduce a new param into LogicalRelation in future. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? N/A (Existing tests would cover it) ### Was this patch authored or co-authored using generative AI tooling? No. -- 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]
