Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator

Поиск
Список
Период
Сортировка
От Roma Sokolov
Тема Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator
Дата
Msg-id F91420DD-0209-44EA-AB37-B22F641EC943@gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator  (Euler Taveira <euler@timbira.com.br>)
Ответы Re: [PATCH] fix DROP OPERATOR to reset links to itself on commutator and negator  (Euler Taveira <euler@timbira.com.br>)
Re: [PATCH] fix DROP OPERATOR to reset links to itself oncommutator and negator  (Yury Zhuravlev <u.zhuravlev@postgrespro.ru>)
Список pgsql-hackers
Дерево обсуждения 
Thanks for comments, Euler!

> ... is hard to understand. Instead, you could separate the conditional
> expression into a variable.

Fixed the patch to be more descriptive and to avoid repeating same
computation over and over again. See v2 of the patch attached.

> I don't think those are mandatory.


Why do you think that? Should I remove them or maybe send as separate
patch?

Cheers,
Roma
Вложения

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

Предыдущее
От: Konstantin Knizhnik
Дата:
Сообщение: Re: The plan for FDW-based sharding
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: [REVIEW] In-core regression tests for replication, cascading, archiving, PITR, etc.