From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
---|---|
To: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: DOCS - create publication (tweak for generated columns) |
Date: | 2025-04-28 02:38:00 |
Message-ID: | CAHut+PsqaWHihhJ17ZSGTD3cqPyGTYPQ6Z-RYN888TruLzDeCg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Apr 23, 2025 at 10:33 AM David G. Johnston
<david(dot)g(dot)johnston(at)gmail(dot)com> wrote:
...
> In the column list I would stick to mentioning what cannot be specified, since it would be assumed by default that any column on the table is fair game. I believe that means not mentioning stored generated and instead mentioning virtual generated. It really seems odds mentioning stored generated being allowed before mentioning when they are optional, default excluded.
>
> Alternative:
>
> The optional column list may include any non-virtual columns of the table. If omitted, tables publish all (including future) non-generated columns by default, and may publish stored generated columns if the publication option publish_generated_columns is set to stored. See Section 29.5 for details about column lists.
>
>
> I don’t get why truncate is mentioned here. I omitted it intentionally.
>
Hi David,
Thanks for your feedback/suggestions. Sorry for my late reply -- I was
waiting for any other feedback, but none was forthcoming, so here is
my patch v2.
I've rearranged the text and mentioned non-virtual and non-generated
as suggested.
I also did not really understand why TRUNCATE is mentioned in this
paragraph, but I have left that as-is for now. For this thread, I only
wanted to address the generated column issue. I think any removal of
the TRUNCATE note ought to be handled separately, where more effort
can be spent to understand why it was put there in the first place.
Anyway, PSA patch v2.
======
Kind Regards,
Peter Smith.
Fujitsu Australia
Attachment | Content-Type | Size |
---|---|---|
v2-0001-DOCS-create-publication-for-table.patch | application/octet-stream | 1.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | jian he | 2025-04-28 02:45:13 | domain over virtual generated column |
Previous Message | Michael Paquier | 2025-04-28 00:12:30 | Re: Avoid circular header file dependency |