Re: [BUGS] BUG #1927: incorrect timestamp returned

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: postgres(at)saparev(dot)com, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [BUGS] BUG #1927: incorrect timestamp returned
Date: 2005-10-08 02:59:33
Message-ID: 11361.1128740373@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-patches

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> Tom Lane wrote:
>> I think it'd probably be better to just fix the rounding during display.

> If we do that, should we remove some the existing JROUND calls in the
> code? I think we have to do this consistently, at least.

Yeah, I was looking at that --- I think most if not all of the existing
JROUND calls ought to go away. Will try to work up a full patch over
the weekend.

> Also, I don't see how rounding is going to fix the problem that the
> value is actually _rounded_ at different stages, meaning when you are
> doing the output you don't know what came in, as outlined by my
> timestamp_in data.

I think the solution is that timestamp_out needs to decide how many
fractional digits it wants to display, and then round off the input
accordingly, *before* it breaks the input down into y/m/d/h/m/s fields.
This "60.00" business is happening because the rounding is done only on
the seconds-and-fractional-seconds field.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Bruce Momjian 2005-10-08 03:06:47 Re: [BUGS] BUG #1927: incorrect timestamp returned
Previous Message Bruce Momjian 2005-10-08 02:46:44 Re: [BUGS] BUG #1927: incorrect timestamp returned

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2005-10-08 03:06:47 Re: [BUGS] BUG #1927: incorrect timestamp returned
Previous Message Bruce Momjian 2005-10-08 02:46:44 Re: [BUGS] BUG #1927: incorrect timestamp returned