Hi, On Tue, Sep 7, 2010 at 6:16 AM, Alvaro Herrera <alvhe...@commandprompt.com> wrote: > Excerpts from Jim Nasby's message of jue jun 10 17:54:43 -0400 2010: >> test...@workbook=# select has_table_privilege( 'public', 'test', 'SELECT' ); >> ERROR: role "public" does not exist > > Here's a patch implementing this idea.
It specially treats only "public" in all lower cases, right? The pseudo-role name is described as "PUBLIC" (upper) in docs, but we accept only "public" (lower) as the pseudo-name. BTW, does the patch need to be back-patched to older versions? Since they use get_roleid_checked() instead of get_role_oid(), the fix cannot be applied cleanly to them, though it will be similar codes. -- Itagaki Takahiro -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
