Re: Review: GiST support for UUIDs

Ildus Kurbangaliev <i.kurbangaliev@postgrespro.ru>

From: Ildus Kurbangaliev <i.kurbangaliev@postgrespro.ru>
To: Paul Jungwirth <pj@illuminatedcomputing.com>
Cc: Teodor Sigaev <teodor@sigaev.ru>, Pgsql Hackers <pgsql-hackers@postgresql.org>
Date: 2015-12-24T12:32:09Z
Lists: pgsql-hackers

Attachments

On Wed, 23 Dec 2015 16:36:23 -0800
Paul Jungwirth <pj@illuminatedcomputing.com> wrote:

> On 12/23/2015 08:10 AM, Ildus Kurbangaliev wrote:
> > There is a more improved version of the patch. Main idea is to
> > present uuid as two uint64 values, and make comparisons and penalty
> > calculation based on these values. This approach is much faster
> > than using memcmp for uuid comparisons.  
> 
> Thank you for picking this up! I'm sorry I was not able to work on it 
> the last few months. I'm very glad to see someone wrapping it up. I'm 
> not a reviewer, but personally it looks like a good change to me.
> 
> Happy holidays,
> 
> Paul
> 
> 
> 
> 

Thanks! The patch was almost done and ready. I attached new version of
the patch with compability changes.

-- 
Ildus Kurbangaliev
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company

Commits

  1. Add uuid to the set of types supported by contrib/btree_gist.