Applied.
---
Bruce Momjian wrote:
> Tom Lane wrote:
> > Michael Fuhr <[EMAIL PROTECTED]> writes:
> > > I think this is what Andrus is seeing:
> >
> > > test=> CREATE TABLE foo (n numeric(9,3));
> > > CREATE TABLE
> > > test=> INSERT INTO foo VALUES (100);
> > > ERROR: numeric field overflow
> > > DETAIL: The absolute value is greater than or equal to 10^6 for field
> > > with precision 9, scale 3.
> > > test=> INSERT INTO foo VALUES (10);
> > > ERROR: numeric field overflow
> > > DETAIL: The absolute value is greater than or equal to 10^9 for field
> > > with precision 9, scale 3.
> >
> > Hm, I thought I tested that same case, but I must've messed up somehow.
> >
> > Anyway, the code seems to be intentionally reporting the log10 of the
> > actual input value, not the limiting log10 for the field size. This
> > behavior goes at least as far back as PG 7.0, so I'm disinclined to
> > change it. We could talk about altering the message wording though,
> > if you have a suggestion for something you'd find less confusing.
> > Pre-7.4 versions say
> >
> > ERROR: overflow on numeric ABS(value) >= 10^9 for field with precision 9
> > scale 3
> >
> > so it looks like we just fixed the grammar during the 7.4 message
> > wording cleanup, without reflecting about whether the meaning was clear.
>
> Yes, this message clearly needs help. Here is what I developed:
>
> test=> CREATE TABLE foo (n numeric(9,3));
> CREATE TABLE
> test=> INSERT INTO foo VALUES (1000);
> ERROR: numeric field overflow
> DETAIL: A field with precision 9, scale 3 must have an absolute value
> less than 10^6.
>
> and the 10^6 is based on the max digits to the left of the decimal
> point, not the input value.
>
> --
> Bruce Momjian| http://candle.pha.pa.us
> pgman@candle.pha.pa.us | (610) 359-1001
> + If your life is a hard drive, | 13 Roberts Road
> + Christ can be your backup.| Newtown Square, Pennsylvania 19073
> Index: src/backend/utils/adt/numeric.c
> ===
> RCS file: /cvsroot/pgsql/src/backend/utils/adt/numeric.c,v
> retrieving revision 1.88
> diff -c -c -r1.88 numeric.c
> *** src/backend/utils/adt/numeric.c 22 Nov 2005 18:17:23 - 1.88
> --- src/backend/utils/adt/numeric.c 23 Jan 2006 20:00:54 -
> ***
> *** 3206,3213
> ereport(ERROR,
>
> (errcode(ERRCODE_NUMERIC_VALUE_OUT_OF_RANGE),
>errmsg("numeric field
> overflow"),
> ! errdetail("The
> absolute value is greater than or equal to 10^%d for field with precision %d,
> scale %d.",
> !
> ddigits - 1, precision, scale)));
> break;
> }
> ddigits -= DEC_DIGITS;
> --- 3206,3213
> ereport(ERROR,
>
> (errcode(ERRCODE_NUMERIC_VALUE_OUT_OF_RANGE),
>errmsg("numeric field
> overflow"),
> ! errdetail("A field
> with precision %d, scale %d must have an absolute value less than 10^%d.",
> !
> precision, scale, maxdigits)));
> break;
> }
> ddigits -= DEC_DIGITS;
>
> ---(end of broadcast)---
> TIP 5: don't forget to increase your free space map settings
--
Bruce Momjian| http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup.| Newtown Square, Pennsylvania 19073
---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to [EMAIL PROTECTED] so that your
message can get through to the mailing list cleanly
