On Thursday 21 April 2005 15:57, Tom Lane wrote:
> I don't think it's a new bug, seeing that I didn't change the code
> for the ISO case.
>
> I see the imprecise result only in the non-integer-datetime case; is
> it acting differently for you?
You're right - it has nothing to do with your patch. I
Tom Lane wrote:
> Oliver Siegmar <[EMAIL PROTECTED]> writes:
> > On Thursday 21 April 2005 15:57, Tom Lane wrote:
> >> If it is only the float case, some imprecision is to be expected.
>
> > So everything is okay?
>
> Well, it's not necessarily *wrong*, but maybe we could improve it.
> The code c
Oliver Siegmar <[EMAIL PROTECTED]> writes:
> On Thursday 21 April 2005 15:57, Tom Lane wrote:
>> If it is only the float case, some imprecision is to be expected.
> So everything is okay?
Well, it's not necessarily *wrong*, but maybe we could improve it.
The code currently assumes it can print 10
Oliver Siegmar <[EMAIL PROTECTED]> writes:
> It removed the bug, but also added a new one (hopefully only one ;-))
I don't think it's a new bug, seeing that I didn't change the code
for the ISO case.
I see the imprecise result only in the non-integer-datetime case; is
it acting differently for yo
> I've applied this patch.
It removed the bug, but also added a new one (hopefully only one ;-))
...now with ISO DateStyle -
select '2005 years 4 mons 20 days 15 hours 57 mins 12.1 secs ago'::interval;
Before your patch:
interval
---
Hi Tom,
On Wednesday 20 April 2005 17:57, Tom Lane wrote:
> "Oliver Siegmar" <[EMAIL PROTECTED]> writes:
> > select '10.10 secs ago'::interval;
> >
> > interval
> > ---
> > @ 10.-10 secs ago
> > (1 row)
>
> What datestyle are you using?
Non-ISO (Postgres in that case), but t
I've applied this patch.
regards, tom lane
Index: datetime.c
===
RCS file: /cvsroot/pgsql/src/backend/utils/adt/datetime.c,v
retrieving revision 1.137
diff -c -r1.137 datetime.c
*** datetime.c 11 Jan 2005 18:
Oliver Siegmar <[EMAIL PROTECTED]> writes:
>> What datestyle are you using?
> Non-ISO (Postgres in that case), but the handling for non-ISO is all the same
> in interval.c ...
Yeah, I just confirmed here that it's broken the same way in all three
non-ISO datestyles. Will look into a fix later to
"Oliver Siegmar" <[EMAIL PROTECTED]> writes:
> select '10.10 secs ago'::interval;
> interval
> ---
> @ 10.-10 secs ago
> (1 row)
What datestyle are you using?
regards, tom lane
---(end of broadcast)---
The following bug has been logged online:
Bug reference: 1609
Logged by: Oliver Siegmar
Email address: [EMAIL PROTECTED]
PostgreSQL version: 8.0.2
Operating system: Linux
Description:Bug in interval datatype for 64 Bit timestamps
Details:
Postgres compiled with --en
10 matches
Mail list logo