Re: WIP: BRIN multi-range indexes

From: Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andres Freund <andres(at)anarazel(dot)de>, Mark Dilger <hornschnorter(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: WIP: BRIN multi-range indexes
Date: 2019-09-14 17:30:49
Message-ID: 20190914173049.welhipsj7bijqahm@development
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Tue, Sep 03, 2019 at 06:05:04PM -0400, Alvaro Herrera wrote:
>On 2019-Jun-11, Tomas Vondra wrote:
>> Attached is this patch series, rebased on top of current master and the
>> opclass parameters patch [1]. I previously planned to keep those two
>> efforts separate for a while, but I decided to give it a try and the
>> breakage is fairly minor so I'll keep it this way - this patch has zero
>> chance of getting committed with the opclass parameters patch anyway.
>> Aside from rebase and changes due to adopting opclass parameters, the
>> patch is otherwise unchanged.
>This patch series doesn't apply, but I'm leaving it alone since the
>brokenness is the opclass part, for which I have pinged the other

Attached is an updated version of this patch series, rebased on top of
the opclass parameter patches, shared by Nikita a couple of days ago.
There's one extra fixup patch, addressing a bug in those patches.

Firstly, while I have some comments on the opclass parameters patches
(shared in the other thread), I think that patch series is moving in the
right direction. After rebase the code is somewhat simpler and easier to
read, which is good. I'm sure there's some more work needed on the APIs
and so on, but I'm optimistic about that.

The rest of this patch series (0007-0011) is mostly unchanged. I've
fixed a couple of bugs and added some comments (particularly to the
bloom opclass), but that's about it.


Tomas Vondra
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment Content-Type Size
0001-Introduce-opclass-parameters.patch.gz application/gzip 21.0 KB
0002-Introduce-amattoptions.patch.gz application/gzip 1.9 KB
0003-Use-amattoptions-in-contrib-bloom.patch.gz application/gzip 2.7 KB
0004-Use-opclass-parameters-in-GiST-tsvector_ops.patch.gz application/gzip 7.1 KB
0005-Remove-pg_index.indoption.patch.gz application/gzip 11.1 KB
0006-fix-amvalidate.patch.gz application/gzip 556 bytes
0007-Pass-all-keys-to-BRIN-consistent-function-at-once.patch.gz application/gzip 4.4 KB
0008-Move-IS-NOT-NULL-checks-to-bringetbitmap.patch.gz application/gzip 2.7 KB
0009-Move-processing-of-NULLs-from-BRIN-support-functions.patch.gz application/gzip 4.5 KB
0010-BRIN-bloom-indexes.patch.gz application/gzip 21.8 KB
0011-BRIN-multi-range-minmax-indexes.patch.gz application/gzip 29.4 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2019-09-14 18:34:21 Re: logical decoding : exceeded maxAllocatedDescs for .spill files
Previous Message Thomas Rosenstein 2019-09-14 16:03:34 Standby Replication and Replication Delay