dilipbiswal commented on a change in pull request #1947:
URL: https://github.com/apache/iceberg/pull/1947#discussion_r559833594
##########
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:
@rdblue Don't we have an issue of the target table has a column named
"_file" ? I was thinking we may need a way to solve it by creating a distinct
co-relation name if _file is existing in the target relation's output ?
----------------------------------------------------------------
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]
