Thank you for reviewing.

At Tue, 31 Jan 2017 11:28:17 +0100, Pavel Stehule <pavel.steh...@gmail.com> 
wrote in <CAFj8pRADxoycR=pqmtdjy0tbryawaq0_xt6t-qbx_u8_rog...@mail.gmail.com>
> I am doing a review of this set of patches:
> 
> 1. There are no problem with patching, compiling - all regress tests passed

Oh! My new check script made from Michael's suggestion finds one
trailing space in the patch. And PUBLICATION/SUBSCRIPTION
conflicats with this so I'll post the reased version.

> 2. tab complete doesn't work well if I am manually writing "create index
> on" - second "ON" is appended - it is a regression

I'll fix it in the version.

> I didn't find any other issues -
> 
> note: not necessary to implement (nice to have) - I miss a support for OR
> REPLACE flag - it is related to LANGUAGE, TRANSFORMATION,  FUNCTION and
> RULE.

regards,

-- 
Kyotaro Horiguchi
NTT Open Source Software Center




-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to