Horton
Menu

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

Tom Lane <tgl@sss.pgh.pa.us>

View thread
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Simon Riggs <simon@2ndQuadrant.com>
Cc: Robert Haas <rhaas@postgresql.org>, pgsql-hackers@postgresql.org
Date: 2010-07-09T15:09:50Z
Lists: pgsql-hackers 
Simon Riggs <simon@2ndQuadrant.com> writes:
> 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.

That seems both pointless and wrong.  Permissions checks should happen
at execution time not plan time.

			regards, tom lane