From daae901d331293144af8dd125ea59892f5ec3c3d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pablo=20Fco=2E=20P=C3=A9rez=20Hidalgo?= Date: Thu, 23 Feb 2017 13:18:06 +0100 Subject: [PATCH] Even though the case class constructor has been made private, the default `apply` factory for the case class remains public so this constraint is useless since anyone can use that factory to create customized instances. --- .../scala/org/apache/spark/sql/catalyst/catalog/interface.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala index 2b3b575b4c06..dd2a196cac4d 100644 --- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala +++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala @@ -322,7 +322,7 @@ case class CatalogStatistics( } -case class CatalogTableType private(name: String) +class CatalogTableType private(val name: String) object CatalogTableType { val EXTERNAL = new CatalogTableType("EXTERNAL") val MANAGED = new CatalogTableType("MANAGED")