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
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 |