From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: IF (NOT) EXISTS in psql-completion |
Date: | 2017-02-01 08:42:54 |
Message-ID: | CAFj8pRAHCwdwe+NRTQ9JrtMO2OdUWtp1demmv_jGBU2tRRs-CQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2017-02-01 9:37 GMT+01:00 Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp
>:
> 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.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.
>
the content of my last mail is a copy my mail from end of December.
Probably lot of changes there.
> > 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 | Andrew Dunstan | 2017-02-01 08:43:52 | Re: CONNECTION LIMIT and Parallel Query don't play well together |
Previous Message | Shinoda, Noriyoshi | 2017-02-01 08:39:41 | Re: Logical Replication and Character encoding |