| From: | vignesh C <vignesh21(at)gmail(dot)com> |
|---|---|
| To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
| Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Shubham Khanna <khannashubham1197(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, Rajendra Kumar Dangwal <dangwalrajendra888(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, euler(at)eulerto(dot)com |
| Subject: | Re: Pgoutput not capturing the generated columns |
| Date: | 2024-11-05 05:48:41 |
| Message-ID: | CALDaNm00OX2yfamejAXzRVZA1_=1CB+DLr71Qtw3bigaGR7e4Q@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, 5 Nov 2024 at 07:55, Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> Hi Vignesh,
>
> Here are my review comments for the v47-0002 (DOCS) patch.
>
> ======
> diff --git a/doc/src/sgml/ddl.sgml b/doc/src/sgml/ddl.sgml
> index 577bcb4b71..a13f19bdbe 100644
> --- a/doc/src/sgml/ddl.sgml
> +++ b/doc/src/sgml/ddl.sgml
> @@ -517,7 +517,8 @@ CREATE TABLE people (
> Generated columns are allowed to be replicated during logical replication
> according to the <command>CREATE PUBLICATION</command> option
> <link linkend="sql-createpublication-params-with-publish-generated-columns">
> - <literal>include_generated_columns</literal></link>.
> + <literal>include_generated_columns</literal></link>. See
> + <xref linkend="logical-replication-gencols"/> for details.
> </para>
> </listitem>
> </itemizedlist>
>
> Previously (in v1-0002) above there was a link to the new gencols
> section ("See XXX for details"), but in v47 that link is no longer
> included. Why not?
Included it now.
> ======
> doc/src/sgml/ref/create_publication.sgml
>
> - lists.
> + lists. See <xref linkend="logical-replication-gencols-howto"/> for more
> + information on the logical replication of generated columns using a
> + column list publication.
> </para>
>
> I don't really think this change is necessary.
>
> The existing paragraph already says "When a column list is specified,
> only the named columns are replicated.", so there is nothing special
> more than that which we really need to say for generated columns.
>
> Also, this paragraph already has a link to the "Column List" chapter
> for more details, so if the user really wants to learn about column
> lists which happen to have generated columns in them, then that's
> where they should look. and there is a link to the new chapter 29.6
> from there.
Removed it.
The v48 version patch attached at [1] has the changes for the same.
Regards,
Vignesh
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bertrand Drouvot | 2024-11-05 05:49:22 | Re: define pg_structiszero(addr, s, r) |
| Previous Message | vignesh C | 2024-11-05 05:47:02 | Re: Pgoutput not capturing the generated columns |