From: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
---|---|
To: | vignesh C <vignesh21(at)gmail(dot)com> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Fix couple of typos |
Date: | 2025-03-18 06:15:47 |
Message-ID: | CAExHW5vJY-C24_zy7thiKn5fZsenTQ1LNBWHtR0h9Q5GjWDARQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Mar 18, 2025 at 8:49 AM vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> On Mon, 17 Mar 2025 at 16:57, Ashutosh Bapat
> <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> >
> > On Mon, Mar 17, 2025 at 4:46 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> > >
> > > On Mon, Mar 17, 2025 at 3:44 PM vignesh C <vignesh21(at)gmail(dot)com> wrote:
> > > >
> > > > While reviewing another patch, I found a couple of typos:
> > > > 1) subid should have been pubid in AlterPublicationOwner and
> > > > AlterPublicationOwner_oid functions.
> > > > 2) Only tuples with XIDs/MXIDs older than the the
> > > > FreezeLimit/MultiXactCutoff are frozen in the common case.
> > > > Should have been"
> > > > Only tuples with XIDs/MXIDs older than the FreezeLimit/MultiXactCutoff
> > > > are frozen in the common case.
> > > >
> > > > The attached patch has the changes for the same.
> > > >
> > >
> > > The changes look good to me.
> >
> > +1
> >
> > However they look unrelated. We should split them in two commits
> > s/subid/pubid/ to one and s/the the/the/ to another. I also verified
> > that this is the only occurrence of "the the" in the code base.
>
> Thanks, the updated v2 has the changes for the same.
LGTM. I would mention the function name, where the typos are fixed, in
the commit message. But that can be done at the time of commit.
--
Best Wishes,
Ashutosh Bapat
From | Date | Subject | |
---|---|---|---|
Next Message | Shubham Khanna | 2025-03-18 06:16:41 | Re: Adding a '--clean-publisher-objects' option to 'pg_createsubscriber' utility. |
Previous Message | David G. Johnston | 2025-03-18 06:10:52 | Re: pg_recvlogical requires -d but not described on the documentation |