Re: point_ops for GiST - Mailing list pgsql-hackers

From Emre Hasegeli
Subject Re: point_ops for GiST
Date
Msg-id CAE2gYzyAF+BYV4m=YJ+XsQ34mWHngStUkd12S=n+df_sQXscOg@mail.gmail.com
Whole thread Raw
In response to Re: point_ops for GiST  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
Responses Re: point_ops for GiST  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
List pgsql-hackers
> This was already fixed for GiST.
> See following discussion
> http://www.postgresql.org/message-id/CAPpHfdvGticGniaj88VCHzHboXJobUhjLm6c09q_Op_u9EoBFg@mail.gmail.com
> and commit
> http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=3c29b196b0ce46662cb9bb7a1f91079fbacbcabb
> "Consistent" method of GiST influences only search and can't lead to corrupt
> indexes. However, "same" method can lead to corrupt indexes.
> However, this is not the reason to not backpatch the changes and preserve
> buggy behaviour; this is the reason to recommend reindexing to users. And it
> was already backpatched.

Fixing it on the opclass is not an option for BRIN.  We designed BRIN
opclasses extensible using extra SQL level support functions and
operators.  It is possible to support point datatype using box vs
point operators.  Doing so would lead to wrong results, because point
operators use FP macros, but box_contain_pt() doesn't.

GiST opclass could be more clean and extensible, if we wouldn't have
those macros.



pgsql-hackers by date:

Previous
From: Alexander Korotkov
Date:
Subject: Re: point_ops for GiST
Next
From: Alexander Korotkov
Date:
Subject: Re: point_ops for GiST