Forum OpenACS Development: timestamp [with time zone] in postgres 7.3

Collapse
1: timestamp [with time zone] in postgres 7.3
Posted by Mark Aufflick on 09/26/03 07:57 AM
I'm sure i missed a discussion already, but i'm sure you all know that as of 7.3, postgres has changed to conform with SQL92 wrt timestamp - so that "timestamp" alone implies "timestamp *without* time zone"

news-aggregator-create.sql needs this finxing, but i wanted to check if there was an agreement that we just wack a "with time zone" on all timestamp columns (since that was the default in 7.2 and is backward compatible) or do we use timestamptz?

whichever, i'll fix news-agg.

other sql files with bare timestamp types:

bug-tracker-drop.sql
categories-drop.sql
category-package.sql
category-tree-package.sql
ecommerce-create.sql
wp-slim-create.sql
wp-slim-drop.sql

Collapse
2: Re: timestamp [with time zone] in postgres 7.3 (response to 1)
Posted by Mark Aufflick on 09/26/03 08:07 AM
and so news-agg will need upgrade scripts as well, yes?

or is 5.0 the first release with news-agg?

Collapse
3: Re: timestamp [with time zone] in postgres 7.3 (response to 1)
Posted by Don Baccus on 09/27/03 12:16 AM
Grrr ... I ran around like crazy fixing these for 4.6, looks like some new stuff snuck in while I wasn't looking.

Yes, in the source they should be defined as you say.

Collapse
4: Re: timestamp [with time zone] in postgres 7.3 (response to 3)
Posted by Mark Aufflick on 09/29/03 06:16 AM
all of the above are fixed except for ecommerce - most are changed, but a few were missed, so i am thinking that may have been on purpose.

i have lodged a bug and will test...