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-07-27 06:42:50
Message-ID: CAJrrPGf7c5rhcMobmZzFoUpdrLJv9TZLau9xVS3HJKtHjOM4fg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Thu, Jul 24, 2014 at 5:59 PM, Asif Naeem <anaeem(dot)it(at)gmail(dot)com> wrote:
> Sorry for being late. Thank you for sharing updated patch, sgml changes
> seems not working i.e.
>
>> postgres=# select max('192.168.1.5', '192.168.1.4');
>> ERROR: function max(unknown, unknown) does not exist
>> LINE 1: select max('192.168.1.5', '192.168.1.4');
>>
>> ^
>> HINT: No function matches the given name and argument types. You might
>> need to add explicit type casts.
>> postgres=# select min('192.168.1.5', '192.168.1.4');
>> ERROR: function min(unknown, unknown) does not exist
>> LINE 1: select min('192.168.1.5', '192.168.1.4');
>>
>> ^
>> HINT: No function matches the given name and argument types. You might
>> need to add explicit type casts.

I didn't get your point. I tested the patch, the sgml changes are working fine.

> I would suggest the following or similar changes e.g.
>
>> doc/src/sgml/func.sgml
>> </indexterm>
>> <function>max(<replaceable
>> class="parameter">expression</replaceable>)</function>
>> </entry>
>> - <entry>any array, numeric, string, or date/time type</entry>
>> + <entry>any inet, array, numeric, string, or date/time type</entry>
>> <entry>same as argument type</entry>
>> <entry>
>> maximum value of <replaceable
>> @@ -12204,7 +12228,7 @@ NULL baz</literallayout>(3 rows)</entry>
>> </indexterm>
>> <function>min(<replaceable
>> class="parameter">expression</replaceable>)</function>
>> </entry>
>> - <entry>any array, numeric, string, or date/time type</entry>
>> + <entry>any inet, array, numeric, string, or date/time type</entry>
>> <entry>same as argument type</entry>
>> <entry>
>> minimum value of <replaceable

Thanks for reviewing the patch.
I missed the above change. Corrected the same in the attached patch.

Regards,
Hari Babu
Fujitsu Australia

Attachment Content-Type Size
inet_agg_v6.patch application/octet-stream 9.2 KB

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message worthy.vii 2014-07-27 18:45:27 BUG #11072: rhel-latest-x86_64 missing from repo
Previous Message rpvoland 2014-07-26 00:44:15 BUG #11039: installation fails when trying to install C++ redistributable

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2014-07-27 07:00:38 Re: B-Tree support function number 3 (strxfrm() optimization)
Previous Message MauMau 2014-07-27 05:57:44 Re: [RFC] Should smgrtruncate() avoid sending sinval message for temp relations