rdblue commented on code in PR #9510:
URL: https://github.com/apache/iceberg/pull/9510#discussion_r1468937382
##########
spark/v3.5/spark-extensions/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/views/ResolvedV2View.scala:
##########
@@ -22,10 +22,12 @@ package org.apache.spark.sql.catalyst.plans.logical.views
import org.apache.spark.sql.catalyst.analysis.LeafNodeWithoutStats
import org.apache.spark.sql.catalyst.expressions.Attribute
import org.apache.spark.sql.connector.catalog.Identifier
+import org.apache.spark.sql.connector.catalog.View
import org.apache.spark.sql.connector.catalog.ViewCatalog
case class ResolvedV2View(
catalog: ViewCatalog,
- identifier: Identifier) extends LeafNodeWithoutStats {
+ identifier: Identifier,
+ view: View) extends LeafNodeWithoutStats {
Review Comment:
I'm not a fan of keeping the view around like this. It's only used in one
case where it shouldn't be because the check should be done in query execution
rather than in job planning. I don't think this change is needed so I'd roll it
back.
--
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.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
