On Wed, Jan 21, 2026 at 7:39 AM Peter Smith <[email protected]> wrote:
>
> Ping. I thought this small DOCS patch had reached some consensus, but
> then the thread went silent for more than 2 weeks.
>
> So, I created a new CF entry [1] marked "ready for committer". Is
> there anything else I need to do to help get this one over the line?

-        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".

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.

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...

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.

Regards,

-- 
Fujii Masao


Reply via email to