Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-16 Thread Bruce Momjian
On Fri, Feb 15, 2013 at 12:12:03PM -0500, Bruce Momjian wrote: > On Thu, Feb 14, 2013 at 07:21:27PM -0500, Tom Lane wrote: > > Bruce Momjian writes: > > > Agreed. The attached patch modifies pg_check_dir() to report about > > > invisible and lost+found directory entries, and give more helpful > >

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-15 Thread Bruce Momjian
On Thu, Feb 14, 2013 at 07:21:27PM -0500, Tom Lane wrote: > Bruce Momjian writes: > > Agreed. The attached patch modifies pg_check_dir() to report about > > invisible and lost+found directory entries, and give more helpful > > messages to the user. > > I'm not terribly thrilled with special-casi

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-14 Thread Bruce Momjian
On Thu, Feb 14, 2013 at 07:21:27PM -0500, Tom Lane wrote: > Bruce Momjian writes: > > Agreed. The attached patch modifies pg_check_dir() to report about > > invisible and lost+found directory entries, and give more helpful > > messages to the user. > > I'm not terribly thrilled with special-casi

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-14 Thread Tom Lane
Bruce Momjian writes: > Agreed. The attached patch modifies pg_check_dir() to report about > invisible and lost+found directory entries, and give more helpful > messages to the user. I'm not terribly thrilled with special-casing 'lost+found' like that, since it's an extremely filesystem-dependen

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-14 Thread Bruce Momjian
On Tue, Feb 5, 2013 at 08:49:17AM -0500, Peter Eisentraut wrote: > On 2/5/13 7:32 AM, Kevin Grittner wrote: > > Sean Chittenden wrote: > > > >> > Currently src/port/pgcheckdir.c will reject non-empty > >> > directories, which is an issue during initdb(1) when PGDATA is > >> > also the mount poin

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Tom Lane
Sean Chittenden writes: > I agree it's not ideal for some filesystems, but being overly protective > doesn't buy us much either, because in some setups, it's entirely acceptable. No, it isn't. As several people have told you already, the idea of letting a mount point be used directly as a data

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Albe Laurenz
Sean Chittenden wrote: > In thinking about it, I like ignoring the hidden directories and failing when > lost+found is present > because, IMO, filesystems where lost+found is going to be present are exactly > the filesystems that > shouldn't have PGDATA located at the top of a mount point. Huh?

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Sean Chittenden
>> Currently src/port/pgcheckdir.c will reject non-empty >> directories, which is an issue during initdb(1) when PGDATA is >> also the mount point for filesystems that support snapshots (e.g. >> ZFS or UFS2). > >> Granted it's not hard to create a subdirectory, initdb there and >> move the content

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Simon Riggs
On 5 February 2013 13:50, Craig Ringer wrote: > On 02/05/2013 08:32 PM, Kevin Grittner wrote: > >> I would rather add a sentence or two to the >> initdb documentation recommending that a cluster not be created at >> a mount point; it should be created in a directory underneath the >> mount point.

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Craig Ringer
On 02/05/2013 08:32 PM, Kevin Grittner wrote: > I would rather add a sentence or two to the > initdb documentation recommending that a cluster not be created at > a mount point; it should be created in a directory underneath the > mount point. That makes a great deal of sense, actually. There's no

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Peter Eisentraut
On 2/5/13 7:32 AM, Kevin Grittner wrote: > Sean Chittenden wrote: > >> > Currently src/port/pgcheckdir.c will reject non-empty >> > directories, which is an issue during initdb(1) when PGDATA is >> > also the mount point for filesystems that support snapshots (e.g. >> > ZFS or UFS2). >> > Granted

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Andrew Dunstan
On 02/05/2013 07:32 AM, Kevin Grittner wrote: Sean Chittenden wrote: Currently src/port/pgcheckdir.c will reject non-empty directories, which is an issue during initdb(1) when PGDATA is also the mount point for filesystems that support snapshots (e.g. ZFS or UFS2). Granted it's not hard to cr

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Sean Chittenden
>> Hello. This was bounced my way via IRC[1] and I'm kicking an updated version >> of the patch upstream for review and committing. >> >> Instead, it seems more correct to simply ignore all directories that begin >> with a dot character. I'm not aware of any special directories exposed by >> fi

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Kevin Grittner
Sean Chittenden wrote: > Currently src/port/pgcheckdir.c will reject non-empty > directories, which is an issue during initdb(1) when PGDATA is > also the mount point for filesystems that support snapshots (e.g. > ZFS or UFS2). > Granted it's not hard to create a subdirectory, initdb there and >

Re: [HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Craig Ringer
On 02/05/2013 04:36 PM, Sean Chittenden wrote: > Hello. This was bounced my way via IRC[1] and I'm kicking an updated version > of the patch upstream for review and committing. > > Instead, it seems more correct to simply ignore all directories that begin > with a dot character. I'm not aware of

[HACKERS] src/ports/pgcheckdir.c - Ignore dot directories...

2013-02-05 Thread Sean Chittenden
Hello. This was bounced my way via IRC[1] and I'm kicking an updated version of the patch upstream for review and committing. Currently src/port/pgcheckdir.c will reject non-empty directories, which is an issue during initdb(1) when PGDATA is also the mount point for filesystems that support sn