Re: Documentation update of wal_retrieve_retry_interval to mention table sync worker - Mailing list pgsql-hackers

From Shlok Kyal
Subject Re: Documentation update of wal_retrieve_retry_interval to mention table sync worker
Date
Msg-id CANhcyEXvkLf76k4vJVEH68-aWM5yz2AHLtpQwp39ZP2uZtS5Zg@mail.gmail.com
Whole thread Raw
In response to Re: Documentation update of wal_retrieve_retry_interval to mention table sync worker  (Peter Smith <smithpb2250@gmail.com>)
List pgsql-hackers
On Mon, 13 Jan 2025 at 12:33, vignesh C <vignesh21@gmail.com> wrote:
>
> On Mon, 6 Jan 2025 at 08:47, Peter Smith <smithpb2250@gmail.com> wrote:
> >
> > Hi Vignesh,
> >
> > Some review comments for your v2 patch.
> >
> > ======
> > doc/src/sgml/logical-replication.sgml
> >
> > AFAICT the only difference you made is changing:
> > FROM "a special kind of apply process"
> > TO "a special kind of table synchronization worker process".
> >
> > There is only ONE kind of tablesync process, so I think saying "a
> > special kind of table synchronization worker process" seems
> > misleading. I also thought maybe it is better to mention that this is
> > PER table.
> >
> > SUGGESTION:
> > ... a special table synchronization worker process per table.
>
> Thanks, the updated v3 version patch has the changes for the same.
>
 Hi Vignesh,

I reviewed the v3 patch. And it looks good to me.

Thanks and Regards,
Shlok Kyal



pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Question about behavior of deletes with REPLICA IDENTITY NOTHING
Next
From: vignesh C
Date:
Subject: Re: POC: enable logical decoding when wal_level = 'replica' without a server restart