On Tue, Nov 16, 2021 at 3:06 AM Daniel Gustafsson <dan...@yesql.se> wrote:
>
> > On 15 Nov 2021, at 19:42, Peter Eisentraut 
> > <peter.eisentr...@enterprisedb.com> wrote:
> >
> > On 15.11.21 10:38, Bharath Rupireddy wrote:
> >>> 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.
> >
> > I find the use of RelationUsesLocalBuffers() confusing in this patch. It 
> > would be clearer to check relpersistence directly in both branches of the 
> > if statement.
>
> Admittedly it didn't bother me, but the more I think about it the more I agree
> with Peter, so +1 on changing.

Done. PSA v9 patch.

Regards,
Bharath Rupireddy.

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

Reply via email to