| 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: | Whole Thread | Raw Message | 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
| 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 |