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 -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
