Re: [PATCH] DefaultACLs

2009/10/6 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> Brendan Jurd <direvus(at)gmail(dot)com> writes:
>> I pulled the latest sources and gave it a whirl.  Things worked as
>> expected in psql, but I was a little surprised when I headed into the
>> documentation.  The first place I visited was Chapter 20 - Database
>> Roles and Privileges, but there was no mention of the default ACLs
>> feature in there.
>
> I looked at that (as well as the material in section 5.6) and concluded
> that it was a once-over-lightly presentation that didn't really need to
> delve into this.

Well 5.6 goes as far as to give mention about WITH GRANT OPTION, so I
don't think we'd be going too deep to give a pointer regarding default
ACLs as well.

I don't think we need a fully detailed explanation of default ACLs
here, just a brief mention and a pointer to the reference page.

> But if you want to work it over, feel free to send in
> a docs patch.

I'll have a play around with it and see if I can come up with wording
that I like.

> Personally I'd like to see less duplication between 5.6
> and 20.3, but I'm not quite sure how to refactor it --- the material is
> arguably somewhat relevant in both places.
>

I hear you about the duplication. 20.3 is so brief that I'm quite
tempted to just remove it and instead have a paragraph referencing
GRANT and REVOKE somewhere in 20.1.

Cheers,
BJ