From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
Cc: | 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: Bug in to_timestamp(). |
Date: | 2016-06-23 17:20:46 |
Message-ID: | CA+TgmoaWTgbtd2RTwVvySMVoOcHmWq66rCCxtVywOOX3FJy4tQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Jun 23, 2016 at 1:12 PM, David G. Johnston
<david(dot)g(dot)johnston(at)gmail(dot)com> wrote:
> to_timestamp with its present behavior is, IMO, a poorly designed function
> that would never be accepted today. Concrete proposals for either fixing or
> deprecating (or both) are welcome. Fixing it should not cause unnecessary
> errors to be raised.
Sheesh. Who put you in charge of this? You basically seem like you
are trying to shut up anyone who supports this change, and I don't
think that's right. Alex's opinion is just as valid as yours -
neither more nor less - and he has every right to express it without
being told by you that his emails are "not constructive".
> My main point is that I'm inclined to deprecate it.
I can almost guarantee that would make a lot of users very unhappy.
This function is widely used.
> My second point is if you are going to use this badly designed function you
> need to protect yourself.
I agree that anyone using this function should test their format
strings carefully.
> My understanding is that is not going to change for 9.6.
That's exactly what is under discussion here.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2016-06-23 17:25:01 | Re: Rethinking representation of partial-aggregate steps |
Previous Message | Robert Haas | 2016-06-23 17:16:03 | Re: Bug in to_timestamp(). |