| From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
|---|---|
| To: | Euler Taveira <euler(at)eulerto(dot)com> |
| Cc: | "shveta(dot)malik(at)gmail(dot)com" <shveta(dot)malik(at)gmail(dot)com>, vignesh C <vignesh21(at)gmail(dot)com>, Dilip Kumar <dilipbalaut(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, shveta malik <shvetamalik(at)gmail(dot)com> |
| Subject: | Re: Include schema-qualified names in publication error messages. |
| Date: | 2026-05-07 12:36:08 |
| Message-ID: | CAA4eK1KL809TGPcK1=bFOs3i44Htxv9Cv88MBiO5taoSchPXGw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, May 7, 2026 at 5:50 PM Euler Taveira <euler(at)eulerto(dot)com> wrote:
>
> On Thu, May 7, 2026, at 7:07 AM, Amit Kapila wrote:
> >
> > That makes sense. I have changed that in the attached and made slight
> > modifications in the comments. How about attached?
> >
>
> One minor suggestion is to rename get_relation_qualified_name to
> get_qualified_relation_name. If we want to reuse this new function
> get_relation_qualified_name into other places, shouldn't we move this function
> to lsyscache.c? There are places like matview.c that could benefit from this new
> function.
>
This suggestion sounds reasonable to me but I suggest instead moving
this new function to rel.h as an inline function or into relcache.c as
those are the places where we take functions with Relation as input.
Additionally, we can name the function as
RelationGetQualifiedRelationName().
--
With Regards,
Amit Kapila.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Euler Taveira | 2026-05-07 12:37:17 | Re: Include schema-qualified names in publication error messages. |
| Previous Message | John Naylor | 2026-05-07 12:29:19 | Re: [PATCH] Fix duplicate errmsg in ALTER TABLE SPLIT PARTITION |