You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/02/15 03:51:45 UTC

[GitHub] [spark] HeartSaVioR commented on pull request #31541: Revert "[SPARK-34209][SQL] Delegate table name validation to the session catalog"

HeartSaVioR commented on pull request #31541:
URL: https://github.com/apache/spark/pull/31541#issuecomment-778919814


   I feel the all inputs about rationalization in here are all valuable (thanks!) and it'd be nice if we can update the JIRA to let JIRA issue be self-explanation on rationalization. Given we decide not to go with revert, we have lost the chance to fill out the commit message.
   
   One thing I don't still get is the R&R of session catalog (from what I can read from conf doc is that "the catalog provides the opportunity to extend V1 to leverage V2 interface", but I guess it's not only used for such purpose), and what are exactly the right expectations between session catalog vs custom catalog... but I'll ask it in dev@ mailing list instead.


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



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