Re: insert throw error when year field len > 4 for timestamptz datatype

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com>, Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, Rushabh Lathia <rushabh(dot)lathia(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: insert throw error when year field len > 4 for timestamptz datatype
Date: 2013-10-02 15:00:30
Message-ID: CA+TgmoYE+YVu4r4fnbme5T8QC7a=QPA7KnQaGStcig31dKwPHw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Oct 1, 2013 at 7:52 PM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> On Fri, Sep 27, 2013 at 10:42:17AM +0000, Haribabu kommi wrote:
>> If the changes are very high to deal all scenarios,
>>
>> I feel it is better do it only in scenarios where the use cases needs it, until
>> it is not confusing users.
>>
>> The rest can be documented.
>>
>> Any other opinions/suggestions welcome.
>
> I have reviewed this patch and it is good. The problem is guessing if a
> number with 5+ digits is YMD, HMS, or a year. I have created a modified
> patch, attached, assumes a 5-digit number is a year, because YMD and HMS
> require at least six digits, and used your date/time test to control the
> other cases. I also added a few more regression tests.

In an ideal world the interpretation of the tokens wouldn't depend on
the order in which they appear. But we don't live in an ideal world,
so maybe this is fine.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2013-10-02 15:03:42 Re: pluggable compression support
Previous Message Robert Haas 2013-10-02 14:56:38 Re: logical changeset generation v6.1