Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms().

On Fri, 2010-07-09 at 11:09 -0400, Stephen Frost wrote:
> > Strangely, I was looking into removing the ExecCheckRTPerms check
> > altogether by forcing plan invalidation when permissions are
> updated.
> > That would be a performance tweak that would render this change
> useless.
> 
> I don't see how you could remove ExecCheckRTPerms..?  It's what
> handles
> all permissions checking for DML (like, making sure you have SELECT
> rights on the table you're trying to query).  I could see forcing plan
> invalidation when permissions are updated, sure, but that doesn't mean
> you can stop doing them altogether anywhere.  Where would you move the
> permissions checking to?  

I apologise, when I said removing the check altogether, I meant removing
from the executor path.

-- Simon Riggs           www.2ndQuadrant.comPostgreSQL Development, 24x7 Support, Training and Services