[GitHub] [spark] cloud-fan commented on a diff in pull request #37287: [WIP] code cleanup for CatalogImpl

2022-07-28 Thread GitBox
cloud-fan commented on code in PR #37287: URL: https://github.com/apache/spark/pull/37287#discussion_r932331297 ## sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala: ## @@ -404,82 +354,69 @@ class CatalogImpl(sparkSession: SparkSession) extends Catalog {

[GitHub] [spark] cloud-fan commented on a diff in pull request #37287: [WIP] code cleanup for CatalogImpl

2022-07-28 Thread GitBox
cloud-fan commented on code in PR #37287: URL: https://github.com/apache/spark/pull/37287#discussion_r932317590 ## sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala: ## @@ -110,53 +108,44 @@ class CatalogImpl(sparkSession: SparkSession) extends Catalog {

[GitHub] [spark] cloud-fan commented on a diff in pull request #37287: [WIP] code cleanup for CatalogImpl

2022-07-28 Thread GitBox
cloud-fan commented on code in PR #37287: URL: https://github.com/apache/spark/pull/37287#discussion_r932311862 ## sql/core/src/main/scala/org/apache/spark/sql/internal/CatalogImpl.scala: ## @@ -78,22 +67,18 @@ class CatalogImpl(sparkSession: SparkSession) extends Catalog {