keypointt commented on a change in pull request #25104: [WIP][SPARK-28341][SQL] 
refine the v2 session catalog config
URL: https://github.com/apache/spark/pull/25104#discussion_r316457679
 
 

 ##########
 File path: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
 ##########
 @@ -111,6 +112,35 @@ class Analyzer(
 
   override protected def defaultCatalogName: Option[String] = 
conf.defaultV2Catalog
 
+  private lazy val v2SessionCatalog = new V2SessionCatalog(catalog, conf)
 
 Review comment:
   minor and not a propose to change, just a quick check that the naming 
`V2SessionCatalog` or `SessionCatalogV2` ? 
   
   I find V2 at the end like `WriteToDataSourceV2` or `CatalogV2Implicits` or 
`DataSourceV2Relation` in current code space, but also sometimes 
`V2SessionCatalog`, so I guess either way is ok?
   

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