Re: interval typmodout is broken

On Mon, Oct 13, 2014 at 07:38:39PM -0400, Bruce Momjian wrote:
> I think the basic problem is that the original author had the idea of
> doing:
> 
>     SELECT INTERVAL (2) '100.9999 seconds';
>      interval
>     ----------
>      00:01:41
> 
> and using (2) in that location as a short-hand when the interval
> precision units were not specified, which seems logical.  However, they
> allowed it even when the units were specified:
> 
>     SELECT INTERVAL (2) '100.9999 seconds' HOUR to SECOND;
>      interval
>     ----------
>      00:01:41
> 
> and in cases where the precision made no sense:
>     
>     SELECT INTERVAL (2) '100.9999 seconds' HOUR to MINUTE;
>      interval
>     ----------
>      00:01:00
> 
> I have created the attached patch which only allows parentheses in the
> first case.  

Patch applied.

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + Everyone has their own god. +