Re: pgsql: doc: restrictions on alter database moving default tablespace

From: David G Johnston <david(dot)g(dot)johnston(at)gmail(dot)com>
To: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: doc: restrictions on alter database moving default tablespace
Date: 2014-12-16 03:54:54
Message-ID: 1418702094165-5830832.post@n5.nabble.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Noah Misch-2 wrote
> On Fri, Dec 12, 2014 at 04:14:50PM -0500, Bruce Momjian wrote:
>> On Tue, Oct 21, 2014 at 01:38:43AM -0400, Noah Misch wrote:
>> > On Sat, Oct 18, 2014 at 02:23:46PM +0000, Bruce Momjian wrote:
>> > > doc: restrictions on alter database moving default tablespace
>> > >
>> > > Mention tablespace must be empty and no one connected to the
>> database.
>> >
>> > The database must not contain any object explicitly assigned to its
>> future
>> > default tablespace, but that tablespace need not be empty. Objects of
>> other
>> > databases in the tablespace pose no problem.
>>
>> I am back to look at this comment, and I don't understand what change is
>> being requested. Are you saying it is just the commit message that
>> isn't clear?
>>
>> The new text is:
>>
>> The new default tablespace for this database must be empty, and no one
>> can be connected to the database.
>>
>> That makes the distinction that only the tablespace for the existing
>> database has to be empty, not the entire tablespace.
>
> No, the commit message accurately describes the change. Your reply helps
> me
> understand your conception of the software's behavior, but I still read
> your
> committed documentation text differently. For "the new default tablespace
> for
> this database" to qualify as "empty", it must contain no object from any
> database. A tablespace is a cross-database, unitary object. Each
> tablespace
> is either empty from every perspective or not empty from any perspective.
> Here are a couple of alternatives for the documentation text:
>
> The new default tablespace must not already contain any of the
> database's
> objects, and no one can be connected to the database.
>
> The database must not contain any object explicitly located in the new
> default tablespace, and no one can be connected to the database.

Drive-by here but wouldn't saying (or adding) "cluster" instead of
"database" solve the confusion?

http://www.postgresql.org/docs/9.3/static/creating-cluster.html

A link to that section may be worth considering...

David J.

--
View this message in context: http://postgresql.nabble.com/pgsql-doc-restrictions-on-alter-database-moving-default-tablespace-tp5823550p5830832.html
Sent from the PostgreSQL - committers mailing list archive at Nabble.com.

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2014-12-16 04:03:35 Re: pgsql: Translation updates
Previous Message Noah Misch 2014-12-16 03:33:12 Re: pgsql: doc: restrictions on alter database moving default tablespace