Re: Include schema-qualified names in publication error messages.

On Tue, Apr 28, 2026 at 9:39 PM shveta malik <shveta(dot)malik(at)gmail(dot)com> wrote:
>
> On Tue, Apr 28, 2026 at 4:34 PM Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
> >
> > Previously, error messages in check_publication_add_relation() only
> > reported the relation name when a table could not be added to a
> > publication or included in an EXCEPT clause. This could be ambiguous
> > in databases where the same relation name exists in multiple schemas.
> >
>
> +1
>
> > This patch updates these error messages to use schema-qualified names,
> > improving the clarity of error reporting for CREATE PUBLICATION and
> > ALTER PUBLICATION commands.
> >
> > This has been discussed on another thread [1]
> >
>
> The patch works well.
>
> I think we can pull out
> 'get_namespace_name_or_temp(RelationGetNamespace(targetrel))' and
> 'RelationGetRelationName(targetrel)' into local variables to reduce
> repetition and make the error paths a bit cleaner.
>
> const char *nspname =
> get_namespace_name_or_temp(RelationGetNamespace(targetrel));
> const char *relname = RelationGetRelationName(targetrel);
>

How about having a dedicated function to return the fully qualified
relation name you want, which can then substitute the single %s.

e.g.
errmsg(errormsg, get_qualified_relname(targetrel)), ...

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