Here's a small patch to cause pg_dump to emit the

scale and precision for NUMERIC type column defs.

Keith Parks

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 6d968919a22b7cb42b6c1f137fbf654f5a7ba0b4..8b097068f6513cb66b3a07438d1a88bfd8831b7b 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -2671,6 +2671,10 @@ dumpTables(FILE *fout, TableInfo *tblinfo, int numTables,
        char                    **parentRels;                   /* list of names of parent relations */
        int                     numParents;
        int                     actual_atts;                    /* number of attrs in this CREATE statment */
+       int32                   tmp_typmod;
+       int                     precision;
+       int                     scale;
+
 
        /* First - dump SEQUENCEs */
        if (tablename)
@@ -2749,6 +2753,18 @@ dumpTables(FILE *fout, TableInfo *tblinfo, int numTables,
                                                                        tblinfo[i].atttypmod[j] - VARHDRSZ);
                                                }
                                        }
+                                       else if (!strcmp(tblinfo[i].typnames[j], "numeric"))
+                                       {
+                                               sprintf(q + strlen(q), "numeric");
+                                               if (tblinfo[i].atttypmod[j] != -1)
+                                               {
+                                                       tmp_typmod = tblinfo[i].atttypmod[j] - VARHDRSZ;
+                                                       precision = (tmp_typmod >> 16) & 0xffff;
+                                                       scale     = tmp_typmod & 0xffff;
+                                                       sprintf(q + strlen(q), "(%d,%d)",
+                                                                               precision, scale);
+                                               }
+                                       }
                                        /* char is an internal single-byte data type;
                                         * Let's make sure we force it through with quotes.
                                         * - thomas 1998-12-13