Re: fix integer datetime division rounding error - Mailing list pgsql-patches

From Rocco Altier
Subject Re: fix integer datetime division rounding error
Date
Msg-id 6E0907A94904D94B99D7F387E08C4F5730ECAC@FALCON.INSIGHT
Whole thread Raw
In response to fix integer datetime division rounding error  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-patches
This fixes the problem for me.

Thanks,
    -rocco

> -----Original Message-----
> From: Bruce Momjian [mailto:pgman@candle.pha.pa.us]
> Sent: Sunday, July 24, 2005 12:37 AM
> To: Andrew Dunstan
> Cc: Patches (PostgreSQL); Rocco Altier
> Subject: Re: [PATCHES] fix integer datetime division rounding error
>
>
> Andrew Dunstan wrote:
> >
> > The attached patch seems to fix the rounding error that is causing
> > regression failures on machines with integer datetimes.
> (Source of error
> > discovered by Andrew@Supernews).ISTM this code needs to be
> given some
> > careful analysis - I know it makes my head spin reading it.
>
> Ah, brilliant!  I knew I was missing something fundamental,
> and the use
> of rint() was it.  Strangely enough, the 8.0 code uses rint() in that
> function, but for floating point intervals, and the code was buggy,
> generating negative time values for division.
>
> Patch attached and applied.  I also improved the interval
> multiplication
> code.
>
> --
>   Bruce Momjian                        |  http://candle.pha.pa.us
>   pgman@candle.pha.pa.us               |  (610) 359-1001
>   +  If your life is a hard drive,     |  13 Roberts Road
>   +  Christ can be your backup.        |  Newtown Square,
> Pennsylvania 19073
>

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: fix integer datetime division rounding error
Next
From: "Rocco Altier"
Date:
Subject: Regression - GNUmakefile - pg_usleep