| From: | Paul A Jungwirth <pj(at)illuminatedcomputing(dot)com> |
|---|---|
| To: | Ildus Kurbangaliev <i(dot)kurbangaliev(at)postgrespro(dot)ru>, Teodor Sigaev <teodor(at)sigaev(dot)ru>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Cc: | Adam Brusselback <adambrusselback(at)gmail(dot)com> |
| Subject: | Re: Review: GiST support for UUIDs |
| Date: | 2016-06-20 05:29:43 |
| Message-ID: | CA+renyVWH2NMWejxDLxMeTeC2cdazKfDsGLHohYs6r-XDeY-Qw@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Feb 5, 2016 at 4:45 PM, Paul Jungwirth
<pj(at)illuminatedcomputing(dot)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
| Attachment | Content-Type | Size |
|---|---|---|
| btree_gist_uuid_6.patch | application/octet-stream | 123.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Masahiko Sawada | 2016-06-20 06:18:56 | Actuall row count of Parallel Seq Scan in EXPLAIN ANALYZE . |
| Previous Message | Amit Kapila | 2016-06-20 05:26:04 | Re: parallel.c is not marked as test covered |