Horton

Re: Review: GiST support for UUIDs

Paul A Jungwirth <pj@illuminatedcomputing.com>

View thread View on postgr.es
From: Paul A Jungwirth <pj@illuminatedcomputing.com>
To: Ildus Kurbangaliev <i.kurbangaliev@postgrespro.ru>, Teodor Sigaev <teodor@sigaev.ru>, Pgsql Hackers <pgsql-hackers@postgresql.org>
Cc: Adam Brusselback <adambrusselback@gmail.com>
Date: 2016-06-20T05:29:43Z
Lists: pgsql-hackers

Attachments

On Fri, Feb 5, 2016 at 4:45 PM, Paul Jungwirth
<pj@illuminatedcomputing.com> wrote:
> On 12/25/2015 03:23 AM, Ildus Kurbangaliev wrote:
>> Thank you for review. Fixed.
> Just wanted to follow up on this and see about getting it added to the next
> commitfest. It looks like Iluds's latest patch (btree_gist_uuid_5.patch)
> doesn't appear on the commitfest page here:
> https://commitfest.postgresql.org/7/332/

I took the btree_gist_uuid_5.patch file from Ildus and updated it so
it would apply on the latest commit. My version is attached as
btree_gist_uuid_6.patch. Since Tom recently changed the SQL signatures
of a few functions (gbt_*_union and gbt_*_same), I did the same with
the new uuid functions. I'd love to see this get accepted next time
around (in September it looks like)!

Thanks,
Paul

Commits

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