Re: Fix number skipping in to_number

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

In response to

Browse pgsql-hackers by date

  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