Re: [PATCHES] [WIP] GUC for temp_tablespaces

2007-05-05 Thread Jaime Casanova
On 5/3/07, Bruce Momjian [EMAIL PROTECTED] wrote: Your patch has been added to the PostgreSQL unapplied patches list at: This is an updated version of the patch. Tom objections: - fd.c is too low level for calling code from commands/tablespace.c. This was fixed adding a second parameter to

Re: [PATCHES] [WIP] GUC for temp_tablespaces

2007-05-05 Thread Bruce Momjian
Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. ---

Re: [PATCHES] Diagnostic functions

2007-05-05 Thread Peter Eisentraut
Heikki Linnakangas wrote: Here's an update of Simon's Heap Page Diagnostic Functions. I now consider it ready to commit. How is this better than pg_filedump? diagnostics is an awfully general name, too. -- Peter Eisentraut http://developer.postgresql.org/~petere/

Re: [PATCHES] [WIP] GUC for temp_tablespaces

2007-05-05 Thread Jaime Casanova
On 5/5/07, Bruce Momjian [EMAIL PROTECTED] wrote: Your patch has been added to the PostgreSQL unapplied patches list at: This is final version of the patch (i hope), at least it fixes the problem i had yesterday. -- regards, Jaime Casanova Programming today is a race between software

Re: [PATCHES] [WIP] GUC for temp_tablespaces

2007-05-05 Thread Bruce Momjian
Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. ---

Re: [PATCHES] Clear up strxfrm() in UTF-8 with locale on Windows

2007-05-05 Thread Magnus Hagander
Magnus Hagander wrote: On Wed, May 02, 2007 at 05:25:39PM -0400, Tom Lane wrote: Magnus Hagander [EMAIL PROTECTED] writes: Given this, perhaps the proper approach should instead be to just check the return value, and go from there? Should be a simple enough patch, something like the attached.

[PATCHES] Enable integer datetimes by default

2007-05-05 Thread Neil Conway
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 objections, I'll apply

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