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

Reply via email to