Re: _USE_32BIT_TIME_T Patch - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: _USE_32BIT_TIME_T Patch
Date
Msg-id 504136D2.7090703@dunslane.net
Whole thread Raw
In response to Re: _USE_32BIT_TIME_T Patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: _USE_32BIT_TIME_T Patch  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On 08/31/2012 03:36 PM, Tom Lane wrote:
> Dave Page <dpage@pgadmin.org> writes:
>> As a side note - I'm not sure why _USE_32BIT_TIME_T was removed in the
>> first place; it was added specifically to avoid this sort of problem,
>> though iirc at the time we were thinking of extensions like Slony and
>> PostGIS being built with Mingw for use with the VC++ built server.
> We removed it when we changed our internal time_t usage to 64 bits:
> http://git.postgresql.org/gitweb/?p=postgresql.git&a=commitdiff&h=cd004067742ee16ee63e55abfb4acbd5f09fbaab
> Possibly that was just a brain fade caused by failing to think about
> the distinction between pg_time_t and system time_t.  However, the
> code has been like that since 8.4, and nobody complained before.
> I share Andrew's unease about whether this issue is fully understood.
>
>             


OTOH, the fact that we used to have it and nothing broke that we know of 
is somewhat reassuring.

I'm not  sure what we need to do to progress on this, especially re the 
back branches.

cheers

andrew





pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: PATCH: pgbench - aggregation of info written into log
Next
From: Tom Lane
Date:
Subject: Re: _USE_32BIT_TIME_T Patch