rdblue commented on a change in pull request #2193:
URL: https://github.com/apache/iceberg/pull/2193#discussion_r568885574



##########
File path: 
spark3-extensions/src/main/scala/org/apache/spark/sql/catalyst/utils/RewriteRowLevelOperationHelper.scala
##########
@@ -197,6 +203,22 @@ trait RewriteRowLevelOperationHelper extends 
PredicateHelper with Logging {
       }
     }
   }
+
+  protected def buildWritePlan(childPlan: LogicalPlan, table: Table): 
LogicalPlan = {

Review comment:
       For a follow up, should we add a rule that does this for Append and 
Overwrite plans as well? That would be nice so that we don't have to wait until 
3.2.0 to get write distribution and ordering for normal writes.




----------------------------------------------------------------
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]

Reply via email to