Re: Updates of SE-PostgreSQL 8.4devel patches (r1710)

From: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>
To: KaiGai Kohei <kaigai(at)kaigai(dot)gr(dot)jp>
Cc: KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>, Stephen Frost <sfrost(at)snowman(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bruce Momjian <bruce(at)momjian(dot)us>, Joshua Brindle <method(at)manicmethod(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Josh Berkus <josh(at)agliodbs(dot)com>, PG Hackers <pgsql-hackers(at)postgresql(dot)org>, Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>
Subject: Re: Updates of SE-PostgreSQL 8.4devel patches (r1710)
Date: 2009-03-11 09:25:31
Message-ID: 49B7838B.6050806@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

KaiGai Kohei wrote:
> Heikki Linnakangas wrote:
>> KaiGai Kohei wrote:
>>> * ACL_SELECT_FOR_UPDATE has same value with ACL_UPDATE, so
>>> SE-PostgreSQL
>>> checks db_table:{update} permission on SELECT ... FOR SHARE OF,
>>> instead of db_table:{lock} permission.
>>
>> This again falls into the category of trying to have more fine-grained
>> permissions than vanilla PostgreSQL has. Just give up on the lock
>> permission, and let it check update permission instead. Yes, it can be
>> annoying that you need update-permission to do SELECT FOR SHARE, but
>> that's an existing problem and not in scope for this patch.
>
> Can I consider the term of "problem" means it can be resolved
> in the future (v8.5, if possible) version?

Sure, a patch to address that in 8.5 would be welcome.

I don't know why it's like that. Maybe no-one has just bothered. Or
maybe it's because of backwards-compatibility or SQL standard
compliance. In any case, it would seem useful to separate them in the
future.

--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2009-03-11 10:09:55 idea, proposal: only preloadable libraries (conditional load)
Previous Message KaiGai Kohei 2009-03-11 09:17:18 Re: Updates of SE-PostgreSQL 8.4devel patches (r1710)