Re: pgsql: Add support of multirange matching to the existing range GiST in

From: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Alexander Korotkov <akorotkov(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Add support of multirange matching to the existing range GiST in
Date: 2020-12-30 08:08:20
Message-ID: CAPpHfdv1Bi1kxStTebSz3A9K=cNdz6dEqisFTikji=xnYspg-g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Wed, Dec 30, 2020 at 10:04 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Tue, Dec 29, 2020 at 11:53:14PM +0300, Alexander Korotkov wrote:
> > On Tue, Dec 29, 2020 at 11:45 PM Alexander Korotkov <akorotkov(at)postgresql(dot)org> wrote:
> >> Add support of multirange matching to the existing range GiST indexes
> >
> > I accidentally squash here a commit which implements GiST multirange
> > indexes. That commit was intended to go separately with its own
> > commit message below. Sorry for that.
>
> The buildfarm got a bit angry after this commit:
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=clam&dt=2020-12-30%2003%3A00%3A11
>
> Here is a diff from member clam, that uses ppc64:
> select count(*) from test_multirange_gist where mr <@
> int4range(10,50);
> - count
> --------
> - 500
> -(1 row)
> -
> +ERROR: unknown multirange operator 4540
>
> This points out to OID_RANGE_MULTIRANGE_CONTAINED_OP.

Thank you for pointing this out. I'll investigate this.

> Please note that you should add a "Discussion:" in those commits to be
> able to track down where this stuff has been discussed.

My oversight, sorry.

------
Regards,
Alexander Korotkov

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Noah Misch 2020-12-30 09:46:51 pgsql: In pg_upgrade cross-version test, handle lack of oldstyle_length
Previous Message Michael Paquier 2020-12-30 07:04:34 Re: pgsql: Add support of multirange matching to the existing range GiST in