On Tue, Mar 15, 2022 at 7:18 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Tue, Mar 15, 2022 at 11:43 AM Masahiko Sawada <sawada.m...@gmail.com> > wrote: > > > > 6. > @@ -1583,7 +1649,8 @@ apply_handle_insert(StringInfo s) > TupleTableSlot *remoteslot; > MemoryContext oldctx; > > - if (handle_streamed_transaction(LOGICAL_REP_MSG_INSERT, s)) > + if (is_skipping_changes() || > > Is there a reason to keep the skip_changes check here and in other DML > operations instead of at one central place in apply_dispatch?
Since we already have the check of applying the change on the spot at the beginning of the handlers I feel it's better to add is_skipping_changes() to the check than add a new if statement to apply_dispatch, but do you prefer to check it in one central place in apply_dispatch? Regards, -- Masahiko Sawada EDB: https://www.enterprisedb.com/