On Sun, May 31, 2009 at 06:32:53PM -0400, Tom Lane wrote:
regression=# select '999'::interval second;
The correct interpretation of the input value is certainly 999 seconds.
Agreed; silent truncation like this is confusing and will lead to
unnecessary bugs in users' code.
the attached patch
Sam Mason s...@samason.me.uk writes:
On Sun, May 31, 2009 at 06:32:53PM -0400, Tom Lane wrote:
There is some case to be made that we should throw error here,
which we could do by putting error tests where the attached patch
has comments suggesting an error test.
With things as they are I
I wrote:
Sam Mason s...@samason.me.uk writes:
On Sun, May 31, 2009 at 06:32:53PM -0400, Tom Lane wrote:
There is some case to be made that we should throw error here,
which we could do by putting error tests where the attached patch
has comments suggesting an error test.
With things as they
On Mon, Jun 1, 2009 at 8:04 PM, Tom Lane t...@sss.pgh.pa.us wrote:
I wrote:
Sam Mason s...@samason.me.uk writes:
On Sun, May 31, 2009 at 06:32:53PM -0400, Tom Lane wrote:
There is some case to be made that we should throw error here,
which we could do by putting error tests where the attached
Robert Haas robertmh...@gmail.com writes:
More to the point, it's also what 8.3.7 does:
Well, no, because the cases at issue are where an interval qualifier
is specified. 8.3 did this:
regression=# select '99 seconds'::interval second;
interval
--
00:00:39
(1 row)
and even more
As I mentioned a bit ago
http://archives.postgresql.org/pgsql-hackers/2009-05/msg01505.php
there seems to be a definite problem still remaining with our handling
of interval literals. To wit, this behavior is absolutely not per spec:
regression=# select '999'::interval second;
interval