Re: Review items for EXCEPT TABLE publication

On Fri, 11 Sept 2026 at 13:57, Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
> I don't really see what is gained by splitting those error messages,
> since the user will get an indeterminate one, and either way they have
> to fix it
>
> IMO, a single unified message could be a simpler/better choice here:
>
> e.g.
>
> ereport(ERROR,
> (errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),
> errmsg("cannot change table \"%s\" to unlogged because it is
> referenced by a publication",
> RelationGetRelationName(rel)),
> errdetail("Unlogged relations cannot be published or excluded via
> an EXCEPT clause."),
> errhint("Remove the table from the publication first.")));

I agree with this approach. A single unified error message is better,
especially since the user needs to remove the table from the
publication either way.

Regards,
Vignesh