On Mon, Nov 15, 2021 at 2:14 PM Daniel Gustafsson <dan...@yesql.se> wrote:
>
> > On 15 Nov 2021, at 09:15, Peter Eisentraut 
> > <peter.eisentr...@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.

+1.

> > 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.

Thanks. Attaching the v8 here again.

Regards,
Bharath Rupireddy.

Attachment: v8-0001-Improve-publication-error-messages.patch
Description: Binary data

Reply via email to