rdblue commented on a change in pull request #1947:
URL: https://github.com/apache/iceberg/pull/1947#discussion_r559846362
##########
File path:
spark3-extensions/src/main/scala/org/apache/spark/sql/catalyst/utils/RewriteRowLevelOperationHelper.scala
##########
@@ -103,6 +103,7 @@ trait RewriteRowLevelOperationHelper extends
PredicateHelper with Logging {
}
private def buildFileFilterPlan(matchingRowsPlan: LogicalPlan): LogicalPlan
= {
+ // TODO: For merge-into make sure _file is resolved only from target table.
Review comment:
I think it should be fine. We should throw an exception if the table has
a `_file` column, but that's something we can do later.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
