Please note that this patch is a correction and replacement for an earlier patch in the queue. The patch accompanying the message
should be removed from the queue and not applied.

The one (originally) attached to this message should be applied.


Michael Glaesemann
grzm myrealbox com

On Jun 5, 2005, at 9:17 AM, Bruce Momjian wrote:

Your patch has been added to the PostgreSQL unapplied patches list at:

It will be applied as soon as one of the PostgreSQL committers reviews
and approves it.

---------------------------------------------------------------------- -----

Michael Glaesemann wrote:

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

The original function was equivalent to

     RETURNS timestamptz
     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

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, skipping... ]

---------------------------(end of broadcast)---------------------------
TIP 2: you can get off all lists at once with the unregister command
   (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])

Reply via email to