Hard to maintain duplication in contain_volatile_functions_not_nextval_walker

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Hard to maintain duplication in contain_volatile_functions_not_nextval_walker
Дата
Msg-id 20160527185853.ziol2os2zskahl7v@alap3.anarazel.de
обсуждение исходный текст
Ответы Re: Hard to maintain duplication in contain_volatile_functions_not_nextval_walker  (Amit Kapila <amit.kapila16@gmail.com>)
Re: Hard to maintain duplication in contain_volatile_functions_not_nextval_walker  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Дерево обсуждения 
Hi,

contain_volatile_functions_walker is duplicated, near entirely, in
contain_volatile_functions_not_nextval_walker.

Wouldn't it have been better not to duplicate, and keep a flag about
ignoring nextval in the context variable?

While at it, couldn't we also fold contain_mutable_functions_walker()
together using a similar technique?

Regards,

Andres

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

Предыдущее
От: Merlin Moncure
Дата:
Сообщение: Re: Allow COPY to use parameters
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: [PROPOSAL] Move all am-related reloption code into src/backend/access/[am-name] and get rid of relopt_kind