Re: [pgsql-patches] [HACKERS] less privileged pl install

2007-01-27 Thread Jeremy Drake
This version of the patch includes documentation changes. Please review... -- The more things change, the more they stay insane.Index: doc/src/sgml/catalogs.sgml === RCS file: /data/local/jeremyd/postgres/cvsuproot/pgsql/doc/src/s

Re: [pgsql-patches] [HACKERS] less privileged pl install

2007-01-26 Thread Jeremy Drake
On Fri, 26 Jan 2007, Jeremy Drake wrote: > On Sat, 27 Jan 2007, Tom Lane wrote: > > > I'd go with GetUserId() in the cases where you're not explicitly > > assigning ownership to the datdba role. AFAIR the assumption that > > languages are owned by BOOTSTRAP_SUPERUSERID was just a kluge to use in

Re: [pgsql-patches] [HACKERS] less privileged pl install

2007-01-26 Thread Jeremy Drake
On Sat, 27 Jan 2007, Tom Lane wrote: > I'd go with GetUserId() in the cases where you're not explicitly > assigning ownership to the datdba role. AFAIR the assumption that > languages are owned by BOOTSTRAP_SUPERUSERID was just a kluge to use in > some bits of code that had to have a notion of a

Re: [pgsql-patches] [HACKERS] less privileged pl install

2007-01-26 Thread Tom Lane
Jeremy Drake <[EMAIL PROTECTED]> writes: > The only difference from this is, that when superuser is required, the > owner of the language is not the superuser who created it, but > BOOTSTRAP_SUPERUSERID. This is because my interpretation was that the > "same behavior as currently" took precedence.

Re: [pgsql-patches] [HACKERS] less privileged pl install

2007-01-26 Thread Jeremy Drake
On Thu, 25 Jan 2007, Jeremy Drake wrote: > On Thu, 25 Jan 2007, Jeremy Drake wrote: > > > I think that an ALTER LANGUAGE OWNER TO is the proper response to these > > things, and unless I hear otherwise I will attempt to add this to my > > patch. > > Here is the patch which adds this. It also allo