From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Oliver Ford <ojford(at)gmail(dot)com> |
Cc: | Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Nathan Wagner <nw+pg(at)hydaspes(dot)if(dot)org> |
Subject: | Re: Fix number skipping in to_number |
Date: | 2017-10-02 12:24:41 |
Message-ID: | AC178086-2286-441C-A257-E3E646B62CD6@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 25 Sep 2017, at 02:52, Nathan Wagner <nw+pg(at)hydaspes(dot)if(dot)org> wrote:
>
> On Thu, Aug 17, 2017 at 12:33:02PM +0100, Oliver Ford wrote:
>
>> Ok I've made that change in the attached v3. I'm not sure as I'm on
>> en_US.UTF-8 locale too. Maybe something Windows specific?
>
> This patch applies against master (8485a25a), compiles, and
> passes a make check.
>
> I tested both on my mac laptop, and my linux server.
>
> If we want this patch, I'd say it's ready for committer. We may want
> (and I can't believe I'm saying this) more discussion as to exactly what
> the strategy for to_number() (and friends) is.
Based on this review, I am moving this patch to the next commitfest and will
update it to Ready for committer. There might, as you say, be more discussion
needed, but due to the lack of extensive such so far I’ll move it for now.
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Maksim Milyutin | 2017-10-02 12:37:17 | Re: Add support for tuple routing to foreign partitions |
Previous Message | Robert Haas | 2017-10-02 12:23:49 | Re: [PATCH] Improve geometric types |