From: Aleksander Alekseev <a.alekseev@postgrespro.ru>
To: Teodor Sigaev <teodor@sigaev.ru>
Cc: pgsql-hackers@postgresql.org, Anastasia Lubennikova <lubennikovaav@gmail.com>
Cc: Peter Geoghegan <pg@heroku.com>
Cc: David Steele <david@pgmasters.net>
Cc: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
Subject: Re: WIP: Covering + unique indexes.
Date: Thu, 30 Mar 2017 19:12:36 +0300
Message-ID: <20170330161235.GA1678@e733.localdomain> (raw)
In-Reply-To: <c1f7c059-597f-44e8-7308-73d76ffd493c@sigaev.ru>
References: <56168952.4010101@postgrespro.ru>
<46e1f13e-6887-3851-a525-f046029341eb@postgrespro.ru>
<20170324124709.22419.51178.pgcf@coridan.postgresql.org>
<c1f7c059-597f-44e8-7308-73d76ffd493c@sigaev.ru>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>
Hi Teodor,
> I had a look on patch and played with it, seems, it looks fine. I splitted
> it to two patches: core changes (+bloom index fix) and btree itself. All
> docs are left in first patch - I'm too lazy to rewrite documentation which
> is changed in second patch.
> Any objection from reviewers to push both patches?
These patches look OK. Definitely no objections from me.
--
Best regards,
Aleksander Alekseev
Attachments:
[application/pgp-signature] signature.asc (832B, ../20170330161235.GA1678@e733.localdomain/2-signature.asc)
download
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.alekseev@postgrespro.ru, teodor@sigaev.ru, lubennikovaav@gmail.com, pg@heroku.com, david@pgmasters.net, peter.eisentraut@2ndquadrant.com
Subject: Re: WIP: Covering + unique indexes.
In-Reply-To: <20170330161235.GA1678@e733.localdomain>
* 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