On Thu, Jan 22, 2026 at 2:24 PM Fujii Masao <masao.fujii@gmail.com> 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
