Re: [PATCH] remove redundant ownership checks

Поиск
Список
Период
Сортировка
От KaiGai Kohei
Тема Re: [PATCH] remove redundant ownership checks
Дата
Msg-id 4B287CE9.606@ak.jp.nec.com
обсуждение исходный текст
Ответ на [PATCH] remove redundant ownership checks  (KaiGai Kohei <kaigai@ak.jp.nec.com>)
Ответы Re: [PATCH] remove redundant ownership checks  (Stephen Frost <sfrost@snowman.net>)
Список pgsql-hackers
The patch was not attached...

(2009/12/16 15:15), KaiGai Kohei wrote:
> It is a cleanup patch apart from SELinux and security framework.
>
> Now, EnableDisableRule() checks ownership of the relation which
> owns the rewrite rule to be enabled/disabled.
>
> But it has the following call path, and this check is already done
> in the ATPrepCmd().
>
>   ATExecCmd()
>    ->  ATExecEnableDisableRule()
>     ->  EnableDisableRule()
>
> This patch removes redundant permission checks.
> No need to check same things twice.
>
> Also see the related discussions:
>    http://archives.postgresql.org/pgsql-hackers/2009-09/msg01593.php
>    http://archives.postgresql.org/pgsql-hackers/2009-09/msg01839.php
>    http://archives.postgresql.org/pgsql-hackers/2009-09/msg01840.php
>
> Thanks,


--
OSS Platform Development Division, NEC
KaiGai Kohei <kaigai@ak.jp.nec.com>

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: KaiGai Kohei
Дата:
Сообщение: [PATCH] remove redundant ownership checks
Следующее
От: Pavel Stehule
Дата:
Сообщение: Re: idea - new aggregates median, listagg