Re: [BUGS] BUG #9652: inet types don't support min/max

From: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
To: Asif Naeem <anaeem(dot)it(at)gmail(dot)com>
Cc: Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [BUGS] BUG #9652: inet types don't support min/max
Date: 2014-08-24 02:38:06
Message-ID: CAJrrPGfHgWYfZ5TF+D69duyrknKak2a96=vriURX-nFvvhD6mQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Mon, Aug 18, 2014 at 8:12 PM, Asif Naeem <anaeem(dot)it(at)gmail(dot)com> wrote:
> Thank you for sharing updated patch. With latest 9.5 source code, patch
> build is failing with following error message i.e.
>
>> /Applications/Xcode.app/Contents/Developer/usr/bin/make -C catalog
>> schemapg.h
>> cd ../../../src/include/catalog && '/opt/local/bin/perl' ./duplicate_oids
>> 3255
>> make[3]: *** [postgres.bki] Error 1
>> make[2]: *** [submake-schemapg] Error 2
>> make[1]: *** [all-backend-recurse] Error 2
>> make: *** [all-src-recurse] Error 2
>
>
> New function is being added by following commit i.e.
>
>> commit b34e37bfefbed1bf9396dde18f308d8b96fd176c
>> Author: Robert Haas <rhaas(at)postgresql(dot)org>
>> Date: Thu Aug 14 12:09:52 2014 -0400
>> Add sortsupport routines for text.
>> This provides a small but worthwhile speedup when sorting text, at
>> least
>> in cases to which the sortsupport machinery applies.
>> Robert Haas and Peter Geoghegan
>
>
> It seems that you need to use another oid. Other than this patch looks good
> to me, please share updated patch and feel free to assign it to committer.
> Thanks.

Thanks for your review. Please find the rebased patch to latest HEAD.

Regards,
Hari Babu
Fujitsu Australia

Attachment Content-Type Size
inet_agg_v8.patch application/octet-stream 7.9 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Kevin Grittner 2014-08-24 11:45:39 Re: Postgresql concern of effect of invalid index
Previous Message Matheus de Oliveira 2014-08-23 16:12:49 Re: Postgresql concern of effect of invalid index

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2014-08-24 03:59:44 Missing comment block at the top of streamutil.h and receivelog.h
Previous Message Craig Ringer 2014-08-24 02:34:42 Re: Parallel Sequence Scan doubts