On Wed, Apr 20, 2011 at 9:59 AM, Tom Lane <t...@sss.pgh.pa.us> wrote: > Shigeru Hanada <han...@metrosystems.co.jp> writes: >> Attached patch implements along specifications below. It also includes >> documents and regression tests. Some of regression tests might be >> redundant and removable. > >> 1) "GRANT privilege [(column_list)] ON [TABLE] TO role" also work for >> foreign tables as well as regular tables, if specified privilege was >> SELECT. This might seem little inconsistent but I feel natural to use >> this syntax for SELECT-able objects. Anyway, such usage can be disabled >> with trivial fix. > > It seems really seriously inconsistent to do that at the same time that > you make other forms of GRANT treat foreign tables as a separate class > of object. I think if they're going to be a separate class of object, > they should be separate, full stop. Making them just mostly separate > will confuse people no end.
I agree. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
