Hello.
On Friday 10 July 2015 15:46:35 you wrote:
> * I think it'd be better to use NONE instead of NULL above, to remove
> the estimator. It seems inconsistent when we've used NONE to mean
> "missing" earlier in the same statement.
Ok, you right.
> * The way you check for the NULL in OperatorUpd is wrong. It cannot
> distinguish between a quoted "null", meaning a function called "null",
> and a NULL, meaning no function.
With "none" has a similar problem. I need to correct the grammar.
>
> Attached is a new version of your patch, rebased over current master,
> and the docs formatting fixes.
Thanks. I hope to send the new patch on Monday.
--
Uriy Zhuravlev
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
