Re: Logical replication - initial data synchronization

From: Peter Smith <smithpb2250(at)gmail(dot)com>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Koen De Groote <kdg(dot)dev(at)gmail(dot)com>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, pgsql-docs(at)lists(dot)postgresql(dot)org
Subject: Re: Logical replication - initial data synchronization
Date: 2024-12-16 06:09:58
Message-ID: CAHut+PtjA706tpLr4WQHGhPV2XcCw_hjPdhvuAqzEY54m6Ch2w@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs

On Mon, Nov 25, 2024 at 11:55 AM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> On Fri, Nov 22, 2024 at 9:15 AM Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> >
> > On Thu, Oct 31, 2024 at 06:52:34PM -0400, Bruce Momjian wrote:
> > > On Fri, Nov 1, 2024 at 09:07:38AM +1100, Peter Smith wrote:
> > > > On Fri, Nov 1, 2024 at 2:16 AM Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > > > >
> > > > > On Fri, Oct 25, 2024 at 11:48:16PM +0200, Koen De Groote wrote:
> > > > > > Number 2 would be perfect, I think. It leaves no doubt at all.
> > > > >
> > > > > I went with even more direct wording, patch attached. Glad we could
> > > > > improve this together --- the feedback from everyone was helpful.
> > > > >
> > > >
> > > > One small nit -- I think "PostgreSQL" typically would have SGML markup:
> > > >
> > > > <productname>PostgreSQL</productname>
> > >
> > > Yes, I thought so too, but if you look at the SGML, it uses PostgreSQL
> > > with no markup in many places close to this sentence, e.g. "PostgreSQL
> > > supports both mechanisms concurrently".
> >
> > Patch applied to master. :-)
> >
>
> Sorry for the late nit, but when I read the re-worded text that was
> pushed [1], I felt the word "typically" was misplaced.
>
> Maybe it is a personal opinion, but OTOH Chat-GPT has the same opinion as me:
>
> ------
> Me:
> Is "typically" in the correct place below?
> When logical replication of a table typically starts, PostgreSQL takes
> a snapshot of the table's data on the publisher database and copies it
> to the subscriber.
>
> ChatGPT said:
> The placement of "typically" in your sentence is grammatically
> correct, but it may cause slight confusion about what "typically" is
> modifying. It seems to suggest that the timing of when logical
> replication starts varies, which might not be the intended meaning.
>
> If you mean that it is the process of taking a snapshot that typically
> happens (not the timing of when replication starts), then "typically"
> would be better placed after "PostgreSQL." Here's a revised version
> for clarity:
>
> "When logical replication of a table starts, PostgreSQL typically
> takes a snapshot of the table's data on the publisher database and
> copies it to the subscriber."
>
> This makes it clear that "typically" refers to the snapshot-taking
> process, not the timing of replication's start.
> ------
>
> ======
> [1] https://github.com/postgres/postgres/commit/4c4aaa19a6fed39e0eb0247625331c3df34d8211
>

Hi Bruce, There was no reply yet to my 3-week-old post above regarding
(what I thought was) the misplaced word "typically" so I am bumping
this thread, just in case that post was accidentally overlooked. OTOH,
if you disagree and/or don't plan to modify it, please let me know so
I can take this thread off my watch list.

======
Kind Regards,
Peter Smith.
Fujitsu Australia

In response to

Browse pgsql-docs by date

  From Date Subject
Next Message sulfinu 2024-12-16 08:01:14 Size limit for NOTIFY payload
Previous Message Fujii Masao 2024-12-15 02:22:54 Re: Clarify old WAL files cannot be removed until they are summarized