| Lists: | pgsql-hackers |
|---|
| From: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
|---|---|
| To: | <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | NO INHERIT |
| Date: | 2007-05-01 21:12:07 |
| Message-ID: | 1178053927.3606.236.camel@silverbirch.site |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
I notice that we have two versions of not INHERITing:
ALTER ROLE meek NOINHERIT earth;
ALTER TABLE meek NO INHERIT earth;
Is there some merit in deciding on just one of these syntaxes? It seems
like we will have to support both the above, but we should encourage
just one common way, just for sanity?
(Yes, I understand the word means totally different thing in each case).
--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com
| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: NO INHERIT |
| Date: | 2007-05-01 21:30:22 |
| Message-ID: | 716.1178055022@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
"Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> I notice that we have two versions of not INHERITing:
> ALTER ROLE meek NOINHERIT earth;
> ALTER TABLE meek NO INHERIT earth;
Where are you reading that?
regards, tom lane
| From: | Gregory Stark <stark(at)enterprisedb(dot)com> |
|---|---|
| To: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
| Cc: | <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: NO INHERIT |
| Date: | 2007-05-01 21:36:30 |
| Message-ID: | 87y7k8dy5d.fsf@oxford.xeocode.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
"Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> (Yes, I understand the word means totally different thing in each case).
Geez, you had me worried. So it's just the spelling that you're noting?
--
Gregory Stark
EnterpriseDB http://www.enterprisedb.com
| From: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
|---|---|
| To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: NO INHERIT |
| Date: | 2007-05-01 21:58:10 |
| Message-ID: | 1178056690.3606.241.camel@silverbirch.site |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
On Tue, 2007-05-01 at 17:30 -0400, Tom Lane wrote:
> "Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> > I notice that we have two versions of not INHERITing:
> > ALTER ROLE meek NOINHERIT earth;
>
> > ALTER TABLE meek NO INHERIT earth;
>
> Where are you reading that?
http://developer.postgresql.org/pgdocs/postgres/sql-alterrole.html
http://developer.postgresql.org/pgdocs/postgres/sql-altertable.html
... and gram.y
--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com
| From: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
|---|---|
| To: | "Gregory Stark" <stark(at)enterprisedb(dot)com> |
| Cc: | <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: NO INHERIT |
| Date: | 2007-05-01 22:28:35 |
| Message-ID: | 1178058515.3606.249.camel@silverbirch.site |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
On Tue, 2007-05-01 at 22:36 +0100, Gregory Stark wrote:
> "Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
>
> > (Yes, I understand the word means totally different thing in each case).
>
> Geez, you had me worried. So it's just the spelling that you're noting?
Yes, the space appears to be mis spelled.
--
Simon Riggs
EnterpriseDB http://www.enterprisedb.com
| From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
|---|---|
| To: | "Simon Riggs" <simon(at)2ndquadrant(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: NO INHERIT |
| Date: | 2007-05-01 22:39:28 |
| Message-ID: | 1691.1178059168@sss.pgh.pa.us |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Lists: | pgsql-hackers |
"Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
> On Tue, 2007-05-01 at 17:30 -0400, Tom Lane wrote:
>> "Simon Riggs" <simon(at)2ndquadrant(dot)com> writes:
>>> I notice that we have two versions of not INHERITing:
>>> ALTER ROLE meek NOINHERIT earth;
>>>
>>> ALTER TABLE meek NO INHERIT earth;
>>
>> Where are you reading that?
> http://developer.postgresql.org/pgdocs/postgres/sql-alterrole.html
> http://developer.postgresql.org/pgdocs/postgres/sql-altertable.html
Oh, you're complaining about the lack of parallelism between ALTER ROLE
and ALTER TABLE. They're not really that parallel anyway:
regression=# ALTER ROLE meek NOINHERIT earth;
ERROR: syntax error at or near "earth"
LINE 1: ALTER ROLE meek NOINHERIT earth;
^
regression=#
It's actually "ALTER ROLE meek NOINHERIT", which is spelled that way
because user/role related options have always been spelled that way:
http://www.postgresql.org/docs/6.4/static/sql-createuser.htm
Maybe we'd do it differently if we were starting over, but I don't
see a point in changing it now.
regards, tom lane