Re: GiST operator class for bool

From: Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com>
To: Andrey Borodin <x4mmm(at)yandex-team(dot)ru>, emre(at)hasegeli(dot)com
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: GiST operator class for bool
Date: 2021-11-06 16:09:05
Message-ID: 784a632a-b2ab-8a31-fdc7-96b4c7f63a0e@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11/3/21 16:18, Tomas Vondra wrote:
> Hi,
>
> I looked at this patch today - it's pretty simple and in pretty good
> shape, I can't think of anything that'd need fixing. Perhaps the test
> might also do EXPLAIN like for other types, to verify the new index is
> actually used. But that's minor enough to handle during commit.
>
>
> I've marked this as RFC and will get it committed in a day or two.
>

Pushed, after adding some simple EXPLAIN to the regression test.

Thanks for the patch!

regards

--
Tomas Vondra
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2021-11-06 16:16:09 Re: lastOverflowedXid does not handle transaction ID wraparound
Previous Message Pavel Stehule 2021-11-06 15:40:55 Re: Schema variables - new implementation for Postgres 15