Re: [COMMITTERS] Using contrib modules in check (Re: pgsql: Fix BRIN to use SnapshotAny during summarization) - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: [COMMITTERS] Using contrib modules in check (Re: pgsql: Fix BRIN to use SnapshotAny during summarization)
Date
Msg-id 20150812185242.GL3040@postgresql.org
Whole thread Raw
In response to Using contrib modules in check (Re: pgsql: Fix BRIN to use SnapshotAny during summarization)  (Christoph Berg <myon@debian.org>)
Responses Re: Re: [COMMITTERS] Using contrib modules in check (Re: pgsql: Fix BRIN to use SnapshotAny during summarization)  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
Tom Lane wrote:

> However, we did learn something valuable from the fact that all the
> -DCLOBBER_CACHE_ALWAYS critters failed on it: per my earlier message,
> brin_page_items() is unsafe against a relcache flush on the index.
> I'll put that on the 9.5 open items list.
>
> (If I were tasked with fixing it, I'd be tempted to rewrite it to do
> all the work in one call and return a tuplestore; the alternative
> seems to be to try to keep the index open across multiple calls,
> which would be a mess.)

Here's a patch doing that.

--
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment

pgsql-hackers by date:

Previous
From: Qingqing Zhou
Date:
Subject: Re: fix oversight converting buf_id to Buffer
Next
From: Andres Freund
Date:
Subject: Re: Raising our compiler requirements for 9.6