Hi Alvaro, On Thu, Mar 17, 2022 at 7:47 PM Alvaro Herrera <alvhe...@alvh.no-ip.org> wrote: > > Split ExecUpdate and ExecDelete into reusable pieces > > Create subroutines ExecUpdatePrologue / ExecUpdateAct / > ExecUpdateEpilogue, and similar for ExecDelete. > > Introduce a new struct to be used internally in nodeModifyTable.c, > dubbed ModifyTableContext, which contains all context information needed > to perform these operations, as well as ExecInsert and others. > > This allows using a different schedule and a different way of evaluating > the results of these operations, which can be exploited by a later > commit introducing support for MERGE. It also makes ExecUpdate and > ExecDelete proper shorter and (hopefully) simpler.
I noticed that we failed to update a comment mentioning a now obsolete parameter of ExecCrossPartitionUpdate(). Attached a patch to fix that and an old typo in the same sentence. -- Amit Langote EDB: http://www.enterprisedb.com
fix-obsolete-comment.patch
Description: Binary data
