RE: Add Nullif case for eval_const_expressions_mutator

Поиск
Список
Период
Сортировка
От Hou, Zhijie
Тема RE: Add Nullif case for eval_const_expressions_mutator
Дата
Msg-id ed9fb4e7024f4144b920cddc19c07de8@G08CNEXMBPEKD05.g08.fujitsu.local
обсуждение исходный текст
Ответ на Re: Add Nullif case for eval_const_expressions_mutator  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы RE: Add Nullif case for eval_const_expressions_mutator
Список pgsql-hackers
> > I notice that there are no Nullif case in eval_const_expression.
> > Since Nullif is similar to Opexpr and is easy to implement, I try add
> > this case in eval_const_expressions_mutator.
>
> I think this patch should be about a tenth the size.  Try modeling it on
> the T_SubscriptingRef-etc case, ie, use ece_generic_processing and then
> ece_evaluate_expr to cover the generic cases.  OpExpr is common enough to
> deserve specially optimized code, but NullIf isn't, so shorter is better.

Thanks for the review.

Attaching v2 patch , which followed the suggestion
to use ece_generic_processing and ece_evaluate_expr to simplify the code.

Please have a check.

Best regards,
houzj




Вложения

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

Предыдущее
От: japin
Дата:
Сообщение: Re: Logical Replication - behavior of ALTER PUBLICATION .. DROP TABLE and ALTER SUBSCRIPTION .. REFRESH PUBLICATION
Следующее
От: Peter Geoghegan
Дата:
Сообщение: Re: Deleting older versions in unique indexes to avoid page splits