Re: GiST operator class for bool

Tomas Vondra <tomas.vondra@enterprisedb.com>

From: Tomas Vondra <tomas.vondra@enterprisedb.com>
To: Andrey Borodin <x4mmm@yandex-team.ru>, emre@hasegeli.com
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2021-11-06T16:09:05Z
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



Commits

  1. Add bool to btree_gist documentation

  2. Fix gist_bool_ops to use gbtreekey2

  3. Add bool GiST opclass to btree_gist