From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Jing Wang <jingwangian(at)gmail(dot)com> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>, Michael Meskes <meskes(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] proposal: Support Unicode host variable in ECPG |
Date: | 2017-11-28 02:03:52 |
Message-ID: | CAB7nPqQNW3yPXqZ2DhDbDu8j7txho--QNpXf5bijuCS3969yxg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, Nov 21, 2017 at 4:02 PM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> The patch does not apply anymore and needs a rebase. There are also a
> bunch of whitespace errors, no documentation in the patch, and it
> seems to me that this patch introduces many concepts so it could be
> broken down into many individual steps. Finally, I strongly recommend
> that you review other's patches. You have two patches, including this
> one, registered into this commit fest but your name is showing nowhere
> as a reviewer. The patch you are proposing here is large, the usual
> recommendation being to review one patch of equal difficulty for each
> patch sent to keep the inflow and outflow of patches balanced.
With a high-level look, this patch requires a bit more work.. I am
unsure as well if that's a feature which would prove to be useful, but
without documentation it is hard to make an opinion of it. I am
marking the patch as returned with feedback.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2017-11-28 02:07:06 | Re: [HACKERS] PATCH: pgbench - break out timing data for initialization phases |
Previous Message | Craig Ringer | 2017-11-28 02:02:28 | Re: [PATCH] Atomic pgrename on Windows |