Re: Pgoutput not capturing the generated columns - Mailing list pgsql-hackers

From Peter Smith
Subject Re: Pgoutput not capturing the generated columns
Date
Msg-id CAHut+PuTRjdwr8i9yooGUAymf4AKpqKrUZocvRtnM+ybL8b6Gw@mail.gmail.com
Whole thread Raw
In response to Pgoutput not capturing the generated columns  (Rajendra Kumar Dangwal <dangwalrajendra888@gmail.com>)
Responses Re: Pgoutput not capturing the generated columns
List pgsql-hackers
On Thu, Oct 31, 2024 at 1:14 PM vignesh C <vignesh21@gmail.com> wrote:
>
> On Thu, 31 Oct 2024 at 04:42, Peter Smith <smithpb2250@gmail.com> wrote:
> >
> > On Thu, Oct 31, 2024 at 3:16 AM vignesh C <vignesh21@gmail.com> wrote:
> > >
> > > On Wed, 30 Oct 2024 at 15:06, Amit Kapila <amit.kapila16@gmail.com> wrote:
> > > >
> > > > On Tue, Oct 29, 2024 at 8:50 PM vignesh C <vignesh21@gmail.com> wrote:
> > > > >
> > > > > Thank you for reporting this issue. The attached v46 patch addresses
> > > > > the problem and includes some adjustments to the comments. Thanks to
> > > > > Amit for sharing the comment changes offline.
> > > > >
> > > >
> > > > Pushed. Kindly rebase and send the remaining patches.
> > >
> > > Thanks for committing this patch, here is a rebased version of the
> > > remaining patches.
> > >
> >
> > Hi,
> >
> > I found that the docs of src/sgml/ddl.sgml [1] are still saying:
> >
> >      <para>
> >       Generated columns are skipped for logical replication and cannot be
> >       specified in a <command>CREATE PUBLICATION</command> column list.
> >      </para>
> >
> > But that is contrary to the new behaviour after the "Replicate
> > generated columns when specified in the column list." commit yesterday
> > [2].
> >
> > It looks like an oversight. I think updating that paragraph should
> > have been included with yesterday's commit.
>
> Thanks for the findings, the attached patch has the changes for the same.
>

LGTM.

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



pgsql-hackers by date:

Previous
From: Noah Misch
Date:
Subject: Re: Inval reliability, especially for inplace updates
Next
From: Jingtang Zhang
Date:
Subject: Re: Introduce new multi insert Table AM and improve performance of various SQL commands with it for Heap AM