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 2019/10/15 00:25:00 UTC
[GitHub] [spark] rdblue commented on issue #26120: [SPARK-29014][SQL]
DataSourceV2: Fix current/default catalog usage
rdblue commented on issue #26120: [SPARK-29014][SQL] DataSourceV2: Fix current/default catalog usage
URL: https://github.com/apache/spark/pull/26120#issuecomment-541983999
I think this should also make the default catalog and the session catalog private in CatalogManager, to ensure that the _only_ catalog accessed from rules is the current catalog. If the default catalog or session catalog is the current catalog, they can (and should) be accessed by getting the current catalog. The default and session catalogs should be internal to CatalogManager.
----------------------------------------------------------------
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