Re: [PATCHES] [subxacts] Fixing TODO items

2004-07-28 Thread Alvaro Herrera
On Tue, Jul 27, 2004 at 01:32:01PM -0400, Tom Lane wrote: Alvaro Herrera [EMAIL PROTECTED] writes: There are some likely controversial changes; the Xid caches, in the first place. No kidding ;-) Do you have any theoretical or practical evidence for the usefulness of the negxids cache?

Re: [PATCHES] USING INDEX TABLESPACE

2004-07-28 Thread Christopher Kings-Lynne
create table test (a integer primary key index tablespace loc); create table test (a integer unique index tablespace loc); create table test (a integer); alter table test add primary key(a) index tablespace loc; create table test (a integer); alter table test add unique(a) index tablespace loc;

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Bruce Momjian
Dave, now that we are nearing beta, I think we need to correct the initdb problem with removing the directory on Win32. Would you code this up as something that sits in /port/dirmod.c and have both initdb and DROP DATABASE call the C routine rather than call rm -r/rmdir? (I think those are the

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Dave Page
-Original Message- From: Bruce Momjian [mailto:[EMAIL PROTECTED] Sent: 28 July 2004 09:29 To: Dave Page Cc: Tom Lane; PostgreSQL-development; [EMAIL PROTECTED] Subject: Re: [HACKERS] Cannot initdb in cvs tip Dave, now that we are nearing beta, I think we need to correct

[PATCHES] Some release notes corrections.

2004-07-28 Thread Brian B.
Here's a patch of things I think may need to be adjusted. Maybe all or some of the changes will prove useful. Thanks, Brian B. --- release.sgml.orig 2004-07-28 00:01:22.620329079 -0400 +++ release.sgml2004-07-28 00:04:15.712796036 -0400 @@ -171,7 +171,7 @@ para Server

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Bruce Momjian
Andrew Dunstan wrote: I wanted to keep a solution that was as native to the OS as possible, but because we can't do that on Win32 and few people like the unix system call to 'rm', it is time to clean it up. One question --- why is there a sleep loop needed for unlink in your patch?

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Tom Lane
Andrew Dunstan [EMAIL PROTECTED] writes: The small wrinkle here is that rmtree needs to make a copy of the file names before it starts removing things. In the backend case that means calling palloc() and friends - am I correct in assuming it is reasonable to do this in whatever context

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Andrew Dunstan
Bruce Momjian wrote: Dave, now that we are nearing beta, I think we need to correct the initdb problem with removing the directory on Win32. Would you code this up as something that sits in /port/dirmod.c and have both initdb and DROP DATABASE call the C routine rather than call rm -r/rmdir? (I

Re: [PATCHES] Some release notes corrections.

2004-07-28 Thread Bruce Momjian
Thanks, applied. --- Brian B. wrote: Here's a patch of things I think may need to be adjusted. Maybe all or some of the changes will prove useful. Thanks, Brian B. [ Attachment, skipping... ]

Re: [PATCHES] Admin functions contrib

2004-07-28 Thread Bruce Momjian
I talked to Tom about this today. First, I want to apologize for running you around in circles in this. I don't think we are giving it the attention it needs because of our schedule. I also think the functionality is drifting into the new features territory and this is also part of the delay

Re: [PATCHES] logger subprocess

2004-07-28 Thread Andreas Pflug
Bruce Momjian wrote: 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. Do not apply. I'm investigating issues under win32. Main

Re: [PATCHES] [HACKERS] Cannot initdb in cvs tip

2004-07-28 Thread Andrew Dunstan
Bruce Momjian wrote: Andrew Dunstan wrote: I wanted to keep a solution that was as native to the OS as possible, but because we can't do that on Win32 and few people like the unix system call to 'rm', it is time to clean it up. One question --- why is there a sleep loop needed for unlink in

Re: [PATCHES] [subxacts] Fixing TODO items

2004-07-28 Thread Alvaro Herrera
On Tue, Jul 27, 2004 at 01:32:01PM -0400, Tom Lane wrote: Alvaro Herrera [EMAIL PROTECTED] writes: There are some likely controversial changes; the Xid caches, in the first place. No kidding ;-) Ok, here is another try. This patch includes both the Xid cache rewrite, source documentation

Re: [PATCHES] pgxs: build infrastructure for extensions v4

2004-07-28 Thread Fabien COELHO
Dear peter, Please find attached another new version of a patch which provides a working infrastructure for pg extensions. I hope it addresses all of Peter's comments. I'll be away for the next 3 weeks, so if minor changes are required it would be best if you could proceed without

Re: [PATCHES] pg_ctl -o option dumps core when processing postmaster arguments...

2004-07-28 Thread Bruce Momjian
Uh, this patch is strange: -#define WHITESPACE \f\n\r\t\v/* as defined by isspace() */ +#define WHITESPACE \f\n\r\t\v\0 /* as defined by isspace() */ They are processed the same by the backend because every string has a trailing null. I think there must be some other bug