On Thu, Jun 12, 2008 at 5:35 PM, Tom Lane <[EMAIL PROTECTED]> wrote: > "Alex Hunsaker" <[EMAIL PROTECTED]> writes: >> Yep, I thought about doing the reverse. Namely Just passing the >> DropStmt to RemoveRelation(s). But then all the permission check >> functions are in utility.c. Splitting those out seemed to be about >> the same as splitting out all the ObjectAddress stuff... > > Well, that might actually be a good approach: try to get ProcessUtility > back down to being just a dispatch switch. It's pretty much of a wart > that we're doing any permissions checking in utility.c at all. Possibly > those functions should be moved to aclchk.c and then used from > RemoveRelation(s) and friends, which would stay where they are but > change API.
Ok Ill work up a patch. Whats that saying about sticking with your first instinct? -- Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches
