rdblue commented on issue #26957: [SPARK-30314] Add identifier and catalog information to DataSourceV2Relation URL: https://github.com/apache/spark/pull/26957#issuecomment-577423876 @yuchenhuo, you're right that the create and replace plans already use Identifier and CatalogPlugin. I'm okay with either solution since `Identifier` is already used in the analyzer. Also, I wouldn't worry about the `TableProvider` path very much. That's a degenerate case that wasn't (and isn't) well defined. Most people will use tables with identifiers. Sometimes those identifiers will need to be extracted from options, but identifiers are the normal case.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org