On Thu, Jan 22, 2026 at 2:24 PM Fujii Masao <[email protected]> wrote:
>
...
>
> -        table, the view definition if the relation is a view, a non-default
> -        <link linkend="sql-altertable-replica-identity">replica
> -         identity</link> setting and the
> +        table, any partition tables, the view definition if the relation is a
> +        view, a non-default <link linkend="sql-altertable-replica-identity">
> +        replica identity</link> setting and the
>
> This part doesn't seem to list all possible objects, so it would be better to
> add wording like "for example" or "such as".

OK. Reworded it like that

>
> If this is meant to be illustrative rather than exhaustive, I'm not sure
> we need to add a new item like "any partition tables" this time here.
>

OK. Removed.

> Regarding whole patch, I'm not clear about whether this patch is worthwhile
> if it only adds phrasing like "such as" or "for example". That said, if some
> users currently interpret the list as exhaustive, this change could help
> clarify the intent...
>

Yes. The patch is not doing much now, but OTOH the reason for this
patch adding the "missing" publication entry in the first place was
precisely because I thought the docs page was was documenting
exhaustive lists; so there might be some merit to clear up that
illusion.

> If we decide to commit this change, it seems fine not to backpatch it,
> since it's more of a documentation clarification than a clear bug fix.
>

Agree.

~~~

PSA v4.

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

Attachment: v4-0001-DOCS-psql-describe-table-can-list-publications.patch
Description: Binary data

Reply via email to