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

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Simon Riggs <simon(at)2ndQuadrant(dot)com>
Cc: Robert Haas <rhaas(at)postgresql(dot)org>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms().
Date: 2010-07-09 15:09:50
Message-ID: 28481.1278688190@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Simon Riggs <simon(at)2ndQuadrant(dot)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

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2010-07-09 16:36:28 Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms().
Previous Message Stephen Frost 2010-07-09 15:09:43 Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms().

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2010-07-09 15:19:17 Re: [v9.1] Add security hook on initialization of instance
Previous Message Stephen Frost 2010-07-09 15:09:43 Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms().