Horton
Menu

Re: Pgoutput not capturing the generated columns

vignesh C <vignesh21@gmail.com>

View thread
From: vignesh C <vignesh21@gmail.com>
To: Peter Smith <smithpb2250@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>, Shubham Khanna <khannashubham1197@gmail.com>, Masahiko Sawada <sawada.mshk@gmail.com>, Rajendra Kumar Dangwal <dangwalrajendra888@gmail.com>, pgsql-hackers@lists.postgresql.org, euler@eulerto.com
Date: 2024-11-04T15:16:25Z
Lists: pgsql-hackers 
On Fri, 1 Nov 2024 at 09:23, Peter Smith <smithpb2250@gmail.com> wrote:
>
> On Thu, Oct 31, 2024 at 3:16 AM vignesh C <vignesh21@gmail.com> wrote:
>
> > Thanks for committing this patch, here is a rebased version of the
> > remaining patches.
> >
>
> Hi Vignesh.
>
> Here are my review comments for the docs patch v1-0002.
>
> ======
> Commit message
>
> 1.
> This patch updates docs to describe the new feature allowing
> replication of generated
> columns. This includes addition of a new section "Generated Column
> Replication" to the
> "Logical Replication" documentation chapter.
>
> ~
>
> That first sentence was correct previously when this patch contained
> *all* the gencols documentation, but now some of the feature docs are
> already handled by previous patches, so the first sentence can be
> removed.
>
> Now patch 0002 is only for adding the new chapter, plus the references to it.
>
> ~
>
> /This includes addition of a new section/This patch adds a new section/

Modified

> ======
> doc/src/sgml/protocol.sgml
>
> 2.
>       <para>
> -      Next, one of the following submessages appears for each column
> (except generated columns):
> +      Next, one of the following submessages appears for each column:
>
> AFAIK this simply cancels out a change from the v1-0001 patch which
> IMO should have not been there in the first place. Please refer to my
> v1-0001 review for the same.

Removed it.

The changes for the same are available at v47 version patch attached
at [1]. I have not included the 0003 patch for now, I will include
once these two patch stabilizes.
[1] - https://www.postgresql.org/message-id/CALDaNm2sNfZoFfqOKq9GAjQZd3isqosij9iHaJjn7oQVmLLNYw%40mail.gmail.com

Regards,
Vignesh

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Doc: Generated column replication.

  2. Rename pubgencols_type to pubgencols in pg_publication.

  3. Doc: Fix column name in pg_publication catalog.

  4. Fix buildfarm failure introduced by commit e65dbc9927.

  5. Change publication's publish_generated_columns option type to enum.

  6. Fix \dRp+ output when describing publications with a lower server version.

  7. Replicate generated columns when 'publish_generated_columns' is set.

  8. Doc: Update the behavior of generated columns in Logical Replication.

  9. Replicate generated columns when specified in the column list.

  10. Doc: Generated columns are skipped for logical replication.