From: | Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru> |
---|---|
To: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, amul sul <sulamul(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com>, Alex Ignatov <a(dot)ignatov(at)postgrespro(dot)ru>, Bruce Momjian <bruce(at)momjian(dot)us>, amul sul <sul_amul(at)yahoo(dot)co(dot)in>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Bug in to_timestamp(). |
Date: | 2018-02-17 09:02:40 |
Message-ID: | 20180217090239.GA14632@arthur.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Feb 14, 2018 at 05:23:53PM +0100, Dmitry Dolgov wrote:
> * On line 52 there is a removed empty line, without any other changes around
Agree, it is unnecessary change. There was the macro there before.
> * On line 177 there is a new commented out line of code. I assume it's not
> an
> explanation or something and we don't need it, am I right?
It explains a node type we deal with. But maybe it is not very useful,
so I removed it.
> And Oracle complains about this:
>
> SELECT to_timestamp('2000 + JUN', ' /') FROM dual
> ORA-01830: date format picture ends before converting entire input string
>
> SELECT to_timestamp('2000 + JUN', ' ') FROM dual
> ORA-01830: date format picture ends before converting entire input string
>
> It's sort of corner case, but anyway maybe you would be interested to handle
> it.
I think it could be fixed by another patch. But I'm not sure that it
will be accepted as well as this patch :).
--
Arthur Zakirov
Postgres Professional: http://www.postgrespro.com
Russian Postgres Company
Attachment | Content-Type | Size |
---|---|---|
0001-to-timestamp-format-checking-v12.patch | text/plain | 12.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fabien COELHO | 2018-02-17 09:20:14 | pgbench - allow to specify scale as a size |
Previous Message | David Rowley | 2018-02-17 08:57:35 | Re: [HACKERS] Runtime Partition Pruning |