public inbox for [email protected]  

help / color / mirror / Atom feed

From: Shlok Kyal <[email protected]>
To: vignesh C <[email protected]>
Cc: Peter Smith <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Documentation update of wal_retrieve_retry_interval to mention table sync worker
Date: Mon, 13 Jan 2025 14:43:34 +0530
Message-ID: <CANhcyEXvkLf76k4vJVEH68-aWM5yz2AHLtpQwp39ZP2uZtS5Zg@mail.gmail.com> (raw)
In-Reply-To: <CALDaNm1_v-+Y3rPYS_c9DRNJGui66LHuc1vAq1d95=y8EiqDkg@mail.gmail.com>
References: <CALDaNm3RxGcD4cDAV5Q0_A4n06F3+AAMpxiyND9Zn0dB86hFmg@mail.gmail.com>
	<CAHut+PtCB9v7D3-vF=dJoZvG-9wUzVrX6P1KOyb2dHgqV3j6Eg@mail.gmail.com>
	<CALDaNm1R5uFeNbEKUsdyuAKp=MxZhj8=DJ3UJK67as4LBW9fAg@mail.gmail.com>
	<CAHut+PsiMLTxOnHLFYbccjL=RY1Ke7xRZOcNLQX51Sv4nUG3qA@mail.gmail.com>
	<CALDaNm1_v-+Y3rPYS_c9DRNJGui66LHuc1vAq1d95=y8EiqDkg@mail.gmail.com>

On Mon, 13 Jan 2025 at 12:33, vignesh C <[email protected]> wrote:
>
> On Mon, 6 Jan 2025 at 08:47, Peter Smith <[email protected]> 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

view thread (7+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Documentation update of wal_retrieve_retry_interval to mention table sync worker
  In-Reply-To: <CANhcyEXvkLf76k4vJVEH68-aWM5yz2AHLtpQwp39ZP2uZtS5Zg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox