Re: Distinguish publication exclusions in object addresses

From: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
To: vignesh C <vignesh21(at)gmail(dot)com>
Cc: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Peter Smith <smithpb2250(at)gmail(dot)com>, shveta malik <shveta(dot)malik(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Distinguish publication exclusions in object addresses
Date: 2026-09-16 08:26:26
Message-ID: 0218CC1B-374D-4997-B9C1-804437AA9A09@gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On Sep 16, 2026, at 15:37, vignesh C <vignesh21(at)gmail(dot)com> wrote:
>
> On Wed, 16 Sept 2026 at 12:16, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>>
>>
>>
>>> On Sep 16, 2026, at 13:50, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>>>
>>> --
>>> With Regards,
>>> Amit Kapila.
>>> <v4-0001-Distinguish-publication-exclusions-in-object-addr.patch>
>>
>> V4 overall looks sold to me. Just a couple of small comments:
>>
>> 1
>> ```
>> -- No entry of either kind. testpub_default publishes nothing.
>> SELECT pg_get_object_address('publication excluded relation',
>> '{public, testpub_tbl1}', '{testpub_default}');
>> ERROR: publication relation "testpub_tbl1" in publication "testpub_default" does not exist
>> ```
>>
>> For this new test, the error message is a little surprising to me. Since the requested object type is "publication excluded relation", I would expect the error message to say something like:
>> ```
>> publication excluded relation "testpub_tbl1" in publication "testpub_default" does not exist
>> ```
>
> I think it would add some unnecessary code complexity in this case.
> Since the underlying issue is simply that the relation is not present
> in the publication, I think the existing generic error message is
> sufficient and should be understandable in the context of publication
> excluded relation.
>

I don’t think the fix would add much complexity, please see the attached diff I tried. This error message is shown to users, so I think improving it is worthwhile.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/

Attachment Content-Type Size
nocfbot_fix_error_msg.diff application/octet-stream 2.7 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message vignesh C 2026-09-16 08:56:48 Re: Distinguish publication exclusions in object addresses
Previous Message Amit Kapila 2026-09-16 08:24:45 Re: Distinguish publication exclusions in object addresses