From: | Michael Glaesemann <grzm(at)myrealbox(dot)com> |
---|---|
To: | pgsql-patches Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | (8.1) to_timestamp correction (epoch to timestamptz) |
Date: | 2004-12-12 13:14:40 |
Message-ID: | C81EBABC-4C3F-11D9-8726-000A95C88220@myrealbox.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Note: This patch is intended for 8.1 (as was the original).
I believe the previous patch I submitted to convert Unix epoch to
timestamptz contains a bug relating to its use of AT TIME ZONE. Please
find attached a corrected patch diffed against HEAD, which includes
documentation.
The original function was equivalent to
CREATE FUNCTION to_timestamp (DOUBLE PRECISION)
RETURNS timestamptz
LANGUAGE SQL AS '
select (
(\'epoch\'::timestamptz + $1 * \'1 second\'::interval)
at time zone \'UTC\'
)
';
The AT TIME ZONE 'UTC' removes the time zone from the timestamptz,
returning timestamp. However, the function is declared to return
timestamptz. The original patch appeared to work, but creating this
equivalent function fails as it doesn't return the declared datatype.
The corrected function restores the time zone with an additional AT
TIME ZONE 'UTC':
CREATE FUNCTION to_timestamp (double precision)
returns timestamptz
language sql as '
select (
(\'epoch\'::timestamptz + $1 * \'1 second\'::interval)
at time zone \'UTC\'
) at time zone \'UTC\'
';
Michael Glaesemann
grzm myrealbox com
Attachment | Content-Type | Size |
---|---|---|
to_timestamp-20041212.diff | application/text | 2.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2004-12-12 15:34:36 | Re: [HACKERS] regression script/makefile exit failure |
Previous Message | Tom Lane | 2004-12-11 23:28:15 | Re: [HACKERS] 8.0.0beta5 FailedAssertion (Crash) when casting composite types |