Re: [PATCHES] Enable integer datetimes by default

2007-05-12 Thread Bruce Momjian
Added to TODO: * Have configure choose integer datetimes by default http://archives.postgresql.org/pgsql-patches/2007-05/msg00046.php --- Tom Lane wrote: Neil Conway [EMAIL PROTECTED] writes: On Sun, 2007-06-05 at

Re: [PATCHES] Enable integer datetimes by default

2007-05-06 Thread Neil Conway
On Sun, 2007-06-05 at 00:20 -0400, Andrew Dunstan wrote: I object to the short notice. I think we need to give people a chance to adjust their configs Sure, I can wait a few days (although if we're going to do this for 8.3, we should do it promptly). On reflection, it might actually be wiser

Re: [PATCHES] Enable integer datetimes by default

2007-05-06 Thread Andrew Dunstan
Peter Eisentraut wrote: Andrew Dunstan wrote: I think we need to give people a chance to adjust their configs, especially on the buildfarm, where those who have currently simply removed the --enable-integer-datetimes setting will need to adjust their configs. Why? They have

Re: [PATCHES] Enable integer datetimes by default

2007-05-06 Thread Andrew Dunstan
Neil Conway wrote: On reflection, it might actually be wiser to delay making this change until the beginning of the 8.4 cycle... I think that's probably true, but I can live with it either way as long as there's enough notice. cheers andrew ---(end of

Re: [PATCHES] Enable integer datetimes by default

2007-05-05 Thread Michael Glaesemann
On May 5, 2007, at 22:28 , Neil Conway wrote: Attached is a patch that enables integer datetimes by default, per recent discussion on -hackers. It makes --enable-integer- datetimes the default, and documents the --disable-integer-datetimes configure option as a means to get the previous

Re: [PATCHES] Enable integer datetimes by default

2007-05-05 Thread Neil Conway
On Sat, 2007-05-05 at 22:49 -0500, Michael Glaesemann wrote: Would it make more sense to have phrase it in the positive sense? i.e., --enable-floating-point-datetimes? I guess that's a bit longer, but it says what you're doing, rather than what you're *not* doing. I think the primary

Re: [PATCHES] Enable integer datetimes by default

2007-05-05 Thread Andrew Dunstan
Neil Conway wrote: Attached is a patch that enables integer datetimes by default, per recent discussion on -hackers. It makes --enable-integer-datetimes the default, and documents the --disable-integer-datetimes configure option as a means to get the previous default behavior. Barring any