Re: contrib/xinetops for 8.1 "patch"

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: "Stephen R(dot) van den Berg" <srb(at)cuci(dot)nl>, pgsql-patches(at)postgresql(dot)org
Subject: Re: contrib/xinetops for 8.1 "patch"
Date: 2006-02-09 22:42:17
Message-ID: 5479.1139524937@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> A larger problem is this:

> test=> SELECT '255.255.255.0'::inet - '1.1.1.1'::inet;
> ?column?
> -----------
> -16843265
> (1 row)

> Should subtraction return int8?

Probably, and for that matter the addition operators should take int8;
on IPV6 data even that's not really wide enough.

> We don't have an unsigned data type. Of course we also have this
> excitement:

> test=> SELECT '255.255.255.0'::inet + 1000000;
> ?column?
> ------------
> 0.15.65.64
> (1 row)

> so we underflow and overflow cleanly. Not great, but it works.

"Cleanly" isn't the adjective I'd use for that. There should be an
overflow error.

regards, tom lane

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2006-02-10 00:36:52 Re: [PATCH] psql formatting patch (round 2)
Previous Message Bruce Momjian 2006-02-09 20:51:54 Re: In pg_dump "no owner" mode don't dump owner names in comments