Re: Logical Replication - improve error message while adding tables to the publication in check_publication_add_relation - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Logical Replication - improve error message while adding tables to the publication in check_publication_add_relation
Date
Msg-id 3DE4CBB6-7C3B-4A99-AFFB-5B40AD48CBF0@yesql.se
Whole thread Raw
In response to Re: Logical Replication - improve error message while adding tables to the publication in check_publication_add_relation  (Peter Eisentraut <peter.eisentraut@enterprisedb.com>)
Responses Re: Logical Replication - improve error message while adding tables to the publication in check_publication_add_relation
List pgsql-hackers
> On 15 Nov 2021, at 09:15, Peter Eisentraut <peter.eisentraut@enterprisedb.com> wrote:

> I think this is not an improvement.  It loses the ability of the caller the
> specify exactly why a relation is not acceptable.


Agreed.

> I think a separate errdetail_relpersistence_not_supported() would be a better
> solution (assuming there are enough callers to make it worth a separate
> function).


I still think that the v8 patch posted earlier is the better option, which
increase granularity of error reporting with a small code footprint.

--
Daniel Gustafsson        https://vmware.com/




pgsql-hackers by date:

Previous
From: Masahiko Sawada
Date:
Subject: Re: parallel vacuum comments
Next
From: Dilip Kumar
Date:
Subject: Re: row filtering for logical replication