spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [spark] rdblue commented on issue #26957: [SPARK-30314] Add identifier and catalog information to DataSourceV2Relation
Date Wed, 22 Jan 2020 22:52:13 GMT
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message