| From: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | pavel(dot)stehule(at)gmail(dot)com |
| Cc: | michael(dot)paquier(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: IF (NOT) EXISTS in psql-completion |
| Date: | 2017-02-01 08:37:48 |
| Message-ID: | 20170201.173748.230472015.horiguchi.kyotaro@lab.ntt.co.jp |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Thank you for reviewing.
At Tue, 31 Jan 2017 11:28:17 +0100, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote in <CAFj8pRADxoycR=PqmtdJy0tBrYAWaQ0_xt6t-QBX_U8_ROGCwQ(at)mail(dot)gmail(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Shinoda, Noriyoshi | 2017-02-01 08:39:41 | Re: Logical Replication and Character encoding |
| Previous Message | Kyotaro HORIGUCHI | 2017-02-01 08:36:23 | Cannot shutdown subscriber after DROP SUBSCRIPTION |