| From: | Simon Riggs <simon(at)2ndQuadrant(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(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 16:36:28 |
| Message-ID: | 1278693388.29736.545.camel@ebony |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
On Fri, 2010-07-09 at 11:07 -0400, Robert Haas 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.
>
> Huh. Obviously, I would have refrained from committing the patch had
> I known that it was going to conflict with work someone else was doing
> in this area, at least until we reached consensus on which way to go
> with it, but since you didn't post about it on -hackers, I had no idea
> that was the case. Sounds like you should probably post your proposal
> and we can discuss what to do in general and also with respect to this
> hook.
Sorry, yes, you couldn't possibly know I was looking at that. I just
meant it was strange those things overlapped.
--
Simon Riggs www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Training and Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2010-07-09 16:51:23 | pgsql: Properly report errno/out-of-disk-space error from pg_upgrade |
| Previous Message | Tom Lane | 2010-07-09 15:09:50 | Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms(). |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2010-07-09 16:51:48 | Re: [COMMITTERS] pgsql: Add a hook in ExecCheckRTPerms(). |
| Previous Message | Tom Lane | 2010-07-09 16:16:42 | Re: Assertion failure in get_attstatsslot() |