Re: [HACKERS] relkind check in DefineIndex

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [HACKERS] relkind check in DefineIndex
Дата
Msg-id CAB7nPqS+tz8KsYFvMvhCAuopOsB1=Yh1V4hq2X5bwiCiOcJzoA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] relkind check in DefineIndex  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
Список pgsql-hackers
On Mon, Oct 16, 2017 at 2:56 PM, Amit Langote
<Langote_Amit_f8@lab.ntt.co.jp> wrote:
> On 2017/10/14 4:32, Robert Haas wrote:
>> On Fri, Oct 13, 2017 at 12:38 PM, Alvaro Herrera
>> <alvherre@2ndquadrant.com> wrote:
>>> The relkind check in DefineIndex has grown into an ugly rats nest of
>>> 'if' statements.  I propose to change it into a switch, as per the
>>> attached.
>>
>> wfm
>
> +1

+1. There is as well CreateTrigger(), analyze_rel() or
ATRewriteCatalogs() that do similar things but those are not using
multiple layers of checks.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

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

Предыдущее
От: Ashutosh Bapat
Дата:
Сообщение: Re: [HACKERS] [POC] hash partitioning
Следующее
От: Magnus Hagander
Дата:
Сообщение: Re: [HACKERS] Determine state of cluster (HA)