From: Anastasia Lubennikova <a.lubennikova@postgrespro.ru>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: [BUG] Error in BRIN summarization
Date: Thu, 13 Aug 2020 13:06:18 +0300
Message-ID: <b4ac7566-f890-6322-ab5b-5c86f5f3edd4@postgrespro.ru> (raw)
In-Reply-To: <20200812195819.GA20725@alvherre.pgsql>
References: <20200812195819.GA20725@alvherre.pgsql>
On 12.08.2020 22:58, Alvaro Herrera wrote:
> On 2020-Aug-12, Alvaro Herrera wrote:
>
>> 'anyvisible' mode is not required AFAICS; reading the code, I think this
>> could also hit REINDEX CONCURRENTLY and CREATE INDEX CONCURRENTLY, which
>> do not use that flag. I didn't try to reproduce it there, though.
>> Anyway, I'm going to remove that Assert() I added.
> So this is what I propose as the final form of the fix.
>
Cool.
This version looks much simpler than mine and passes the tests fine.
--
Anastasia Lubennikova
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: a.lubennikova@postgrespro.ru, alvherre@2ndquadrant.com, pgsql-hackers@lists.postgresql.org
Subject: Re: [BUG] Error in BRIN summarization
In-Reply-To: <b4ac7566-f890-6322-ab5b-5c86f5f3edd4@postgrespro.ru>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox