Review feedback

Author: aokolnychyi

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/RelationResolution.scala

@@ -253,6 +253,7 @@ class RelationResolution(override val catalogManager: CatalogManager)
   private def adaptCachedRelation(cached: LogicalPlan, ref: V2TableReference): LogicalPlan = {
     cached transform {
       case r: DataSourceV2Relation if matchesReference(r, ref) =>
+        V2TableReferenceUtils.validateLoadedTable(r.table, ref)
         r.copy(output = ref.output, options = ref.options)
     }
   }

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/V2TableReference.scala

@@ -48,7 +48,7 @@ import org.apache.spark.util.ArrayImplicits._
  * For instance, temporary views with fully resolved logical plans don't allow schema changes
  * in underlying tables.
  */
-case class V2TableReference private(
+private[sql] case class V2TableReference private(
     catalog: TableCatalog,
     identifier: Identifier,
     options: CaseInsensitiveStringMap,
@@ -75,7 +75,7 @@ case class V2TableReference private(
   }
 }
 
-object V2TableReference {
+private[sql] object V2TableReference {
 
   case class TableInfo(
       columns: Seq[Column],
@@ -103,7 +103,7 @@ object V2TableReference {
   }
 }
 
-object V2TableReferenceUtils extends SQLConfHelper {
+private[sql] object V2TableReferenceUtils extends SQLConfHelper {
 
   def validateLoadedTable(table: Table, ref: V2TableReference): Unit = {
     ref.context match {