Re: [PATCH] btree_gin, add support for uuid, bool, name, bpchar andanyrange types

On 03/30/2018 10:51 PM, Matheus de Oliveira wrote:
> Hi all.
> 
> On Wed, Mar 21, 2018 at 1:47 PM, Tomas Vondra
> <tomas.vondra@2ndquadrant.com <mailto:tomas.vondra@2ndquadrant.com>> wrote:
> 
> 
>     Do you plan to post an updated version of the patch, of what is your
>     response to the points raised last week?
> 
> 
> Very sorry about the long delay. I've been in a long trip, no time to
> look that carefully.
>  
> 
>     I still haven't made my mind regarding usefulness of range opclasses, so
>     I suggest to split the patch into two parts - 0001 for the opclasses
>     we're confident are useful, and 0002 for those extras. The committer
>     then may apply either 0001 or 0001+0002, depending on his judgment.
> 
> 
> I liked the idea. So, follows the patches:
> - 0001-btree_gin-uuid--base.v2.patch - all types but anyrange, and with
> the adjustments on comments you proposed
> - 0002-btree_gin-uuid--anyrange.v2.patch - adding the anyrange type
> (must be applied after 0001)
> 
> Anything else missing?
> 

I don't think so. I've marked it as RFC, but I have no idea if anyone is
going to push it by the end of this commitfest.


regards

-- 
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services