On Tue, Nov 23, 2021 at 4:58 PM Ajin Cherian <itsa...@gmail.com> wrote: > > Attaching a new patchset v41 which includes changes by both Peter and myself. >
In 0003 patch, why is below change required? --- a/src/backend/replication/pgoutput/pgoutput.c +++ b/src/backend/replication/pgoutput/pgoutput.c @@ -1,4 +1,4 @@ -/*------------------------------------------------------------------------- +/*------------------------------------------------------------------------ * * pgoutput.c I suggest at this stage we can combine 0001, 0003, and 0004. Then move pg_dump and psql (describe.c) related changes to 0002 and make 0002 as the last patch in the series. This will help review backend changes first and then we can look at client-side changes. After above, rearrange the code in pgoutput_row_filter(), so that two different checks related to 'rfisnull' (introduced by different patches) can be combined as if .. else check. -- With Regards, Amit Kapila.
