On Tue, Oct 1, 2013 at 7:52 PM, Bruce Momjian <bruce@momjian.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
