Re: [PATCH] DefaultACLs - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: [PATCH] DefaultACLs
Date
Msg-id 20090804172800.GM23840@tamriel.snowman.net
Whole thread Raw
In response to Re: [PATCH] DefaultACLs  (Stephen Frost <sfrost@snowman.net>)
Responses Re: [PATCH] DefaultACLs  (Joshua Tolley <eggyknap@gmail.com>)
Re: [PATCH] DefaultACLs  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
* Stephen Frost (sfrost@snowman.net) wrote:
> * Robert Haas (robertmhaas@gmail.com) wrote:
> > On Wed, Jul 22, 2009 at 11:26 PM, Petr Jelinek<pjmodos@pjmodos.net> wrote:
> > > The docs are not complete but that's up to Stephen, otherwise the patch
> > > should be finished. But I am not the reviewer :)
> >
> > Well, perhaps we had better prod Stephen then, since complete docs are
> > a requirement for commit.
>
> I didn't think the docs posted were terrible, but I am working on
> improving them.

Thanks to Joshua, there weren't really many changes I found for the
docs.  Here they are anyway:

grant.sgml:

+ Replace current privileges with the ones specified using
+ <xref linkend="sql-alterschema" endterm="sql-alterschema-title">.
+ The <literal>WITH GRANT OPTION</literal> parameter is not applicable
+ because it is copied from default privileges.
+ Note: this can actually <emphasis role="bold">revoke</emphasis> some
+ privileges because it clears all existing privileges object has and
+ replaces them with the default ones for the schema in which this object
+ resides.

How about:

Replaces current privileges with the default privileges, as set using
<xref linkend="sql-alterschema" endterm="sql-alterschema-title">, for
this object type in its current schema.
The <literal>WITH GRANT OPTION</literal> parameter is not applicable
because it is copied from default privileges.
Note: This can <emphasis role="bold">revoke</emphasis> privileges!  This
will clear all existing privileges first!  If no default privilege has
been set, the object will have all privileges revoked!

Otherwise, I thought the docs looked pretty good.
Thanks,
    Stephen

pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Alpha Releases: Docs?
Next
From: Josh Berkus
Date:
Subject: Re: Alpha Releases: Docs?