Re: error_severity of brin work item

On Wed, Mar 10, 2021 at 08:24:50AM -0500, David Steele wrote:
> On 12/1/20 5:25 PM, Justin Pryzby wrote:
> > On Tue, Dec 01, 2020 at 03:57:24PM -0300, Alvaro Herrera wrote:
> >
> > > > Another idea is if perform_work_item() were responsible for discarding
> > > > relations which disappear. Currently it does this, which is racy since it
> > > > holds no lock.
> > >
> > > That has the property that it remains contained in autovacuum.c, but no
> > > other advantages I think.
> >
> > It has the advantage that it moves all the try_open stuff out of brin.
> >
> > I started implementing this, and then realized that the try_open stuff *has* to
> > be in the brin_summarize function, to handle the case that someone passes a
> > non-index, since it's SQL exposed.
> > So maybe we should use your LockOid patch now, and refactor in the future if we
> > add additional work-item types.
>
> Thoughts on this, Álvaro? I can see that the first version of this patch was
> not ideal but the rework seems to have stalled. Since it is a bug perhaps it
> would be better to get something in as Justin suggests?
>

Hi Álvaro,

Do you plan to work on this for this CF?

--
Jaime Casanova
Director de Servicios Profesionales
SystemGuards - Consultores de PostgreSQL