[GitHub] [spark] cloud-fan commented on issue #24768: [SPARK-27919][SQL] Add v2 session catalog

[GitBox <gi...@apache.org>]

cloud-fan commented on issue #24768: [SPARK-27919][SQL] Add v2 session catalog
URL: https://github.com/apache/spark/pull/24768#issuecomment-509221552
 
 
   I think this PR does 2 things:
   1. Fix the behavior of table name resolution to always respect the default catalog config.
   2. Support `TableProvider` (v2 table provider) in the builtin hive catalog.
   
   For 1, I have a different opinion about the expected behavior. The new behavior after this PR is still slightly different from what I think of. I opened https://github.com/apache/spark/pull/25077 for broader discussions.
   
   For 2, I do like the idea if v2 session catalog. It's a pretty good way to fallback to the Hive catalog before we migrate it to the new catalog API. I'd like to merge it after we reach a consensus for 1.

----------------------------------------------------------------
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