- Support for relkind = RELKIND_VIEW.
authorPhilip Warner
Fri, 15 Sep 2000 04:35:16 +0000 (04:35 +0000)
committerPhilip Warner
Fri, 15 Sep 2000 04:35:16 +0000 (04:35 +0000)
- Use symbols for tests on relkind (ie. use RELKIND_VIEW, not 'v')
- Fix bug in support for -b option (== --blobs).
- Dump views as views (using 'create view').
- Remove 'isViewRule' since we check the relkind when getting tables.
- Now uses temp table 'pgdump_oid' rather than 'pg_dump_oid' (errors otherwise).
- Added extra param for specifying handling of OID=0 and which typename to output.
- Fixed bug in SQL scanner when SQL contained braces. (in rules)
- Use format_type function wherever possible

src/bin/pg_dump/common.c
src/bin/pg_dump/pg_backup_archiver.h
src/bin/pg_dump/pg_backup_db.c
src/bin/pg_dump/pg_dump.c
src/bin/pg_dump/pg_dump.h

index d7cb9e488a5cb4924d0c19a1210c77526becc7b7..8e1847b8d6af326306091c37fc9c7a18a372e682 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/bin/pg_dump/common.c,v 1.45 2000/08/06 17:50:48 thomas Exp $
+ *   $Header: /cvsroot/pgsql/src/bin/pg_dump/common.c,v 1.46 2000/09/15 04:35:16 pjw Exp $
  *
  * Modifications - 6/12/96 - [email protected] - version 1.13.dhb.2
  *
  *  - Added single. quote to twin single quote expansion for 'insert' string
  *    mode.
  *
+ * Modifications 14-Sep-2000 - [email protected] 
+ * -   Added enum for findTypeByOid to specify how to handle OID and which 
+ *     string to return - formatted type, or base type. If the base type
+ *     is returned then fmtId is called on the string.
+ *
+ *     BEWARE: Since fmtId uses a static buffer, using 'useBaseTypeName' on more
+ *             than one call in a line will cause problems.
+ *
  *-------------------------------------------------------------------------
  */
 
@@ -50,17 +58,32 @@ static int  strInArray(const char *pattern, char **arr, int arr_size);
  */
 
 char *
-findTypeByOid(TypeInfo *tinfo, int numTypes, const char *oid)
+findTypeByOid(TypeInfo *tinfo, int numTypes, const char *oid, OidOptions opts)
 {
    int         i;
 
-   if (strcmp(oid, "0") == 0)
-       return g_opaque_type;
+   if (strcmp(oid, "0") == 0) {
+
+       if ( (opts & zeroAsOpaque) != 0 ) {
+
+           return g_opaque_type;
+
+       } else if ( (opts & zeroAsAny) != 0 ) {
+
+           return "'any'";
+
+       }
+   }
 
    for (i = 0; i < numTypes; i++)
    {
-       if (strcmp(tinfo[i].oid, oid) == 0)
-           return tinfo[i].typname;
+       if (strcmp(tinfo[i].oid, oid) == 0) {
+           if ( (opts & useBaseTypeName) != 0 ) {
+               return fmtId(tinfo[i].typname, false);
+           } else {
+               return tinfo[i].typedefn;
+           }
+       }
    }
 
    /* should never get here */
index 4ee00589d6101f2bf27aca1bba8fb1dce3231289..0abd9a77db6c0f23fd76afee07fbe59f6c41b2f0 100644 (file)
  *
  * Modifications - 28-Jun-2000 - [email protected]
  *
- * Initial version. 
+ *     Initial version. 
+ *
+ * Modifications - 15-Sep-2000 - [email protected]
+ * -   Added braceDepth to sqlparseInfo to handle braces in rule definitions.
  *
  *-------------------------------------------------------------------------
  */
@@ -59,7 +62,7 @@ typedef z_stream *z_streamp;
 
 #define K_VERS_MAJOR 1
 #define K_VERS_MINOR 4 
-#define K_VERS_REV 11 
+#define K_VERS_REV 14 
 
 /* Data block types */
 #define BLK_DATA 1
@@ -124,6 +127,7 @@ typedef struct {
    sqlparseState       state;
    char                lastChar;
    char                quoteChar;
+   int                 braceDepth;
 } sqlparseInfo;
 
 typedef struct _archiveHandle {
index bf6ccffc2a982951e6b8eb8529430cdb1230be97..d606508a36a4c8d80520be8cbbcab0e7effe3fb9 100644 (file)
@@ -483,7 +483,7 @@ int ExecuteSqlCommandBuf(ArchiveHandle* AH, void *qryv, int bufLen)
 
                case SQL_SCAN: /* Default state == 0, set in _allocAH */
 
-                   if (qry[pos] == ';')
+                   if (qry[pos] == ';' && AH->sqlparse.braceDepth == 0)
                    {
                        /* Send It & reset the buffer */
                        /* fprintf(stderr, "    sending: '%s'\n\n", AH->sqlBuf->data); */
@@ -507,7 +507,16 @@ int ExecuteSqlCommandBuf(ArchiveHandle* AH, void *qryv, int bufLen)
                        else if (qry[pos] == '*' && AH->sqlparse.lastChar == '/')
                        {
                            AH->sqlparse.state = SQL_IN_EXT_COMMENT;
+                       } 
+                       else if ( qry[pos] == '(' )
+                       {
+                           AH->sqlparse.braceDepth++;
                        }
+                       else if (qry[pos] == ')')
+                       {
+                           AH->sqlparse.braceDepth--;
+                       }
+
                        AH->sqlparse.lastChar = qry[pos];
                    }
 
index 372ff9be4c87c958d74fa44988049bb389447207..ad3dc94ad6931fca8170f53c586d251aa14def3e 100644 (file)
@@ -22,7 +22,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.164 2000/09/12 04:15:58 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.165 2000/09/15 04:35:16 pjw Exp $
  *
  * Modifications - 6/10/96 - [email protected] - version 1.13.dhb
  *
  *     Fixed minor bug in language dumping code: expbuffres were not being reset.
  *     Fixed version number initialization in _allocAH (pg_backup_archiver.c)
  *
+ * Modifications - 14-Sep-2000 - [email protected] 
+ *     Use symbols for tests on relkind (ie. use RELKIND_VIEW, not 'v')
+ *         Support for relkind = RELKIND_VIEW.
+ *         Fix bug in support for -b option (== --blobs).
+ *     Dump views as views (using 'create view').
+ *     Remove 'isViewRule' since we check the relkind when getting tables.
+ *     Now uses temp table 'pgdump_oid' rather than 'pg_dump_oid' (errors otherwise).
+ *
  *-------------------------------------------------------------------------
  */
 
@@ -124,7 +132,6 @@ static void clearTableInfo(TableInfo *, int);
 static void dumpOneFunc(Archive *fout, FuncInfo *finfo, int i,
            TypeInfo *tinfo, int numTypes);
 static int findLastBuiltinOid(void);
-static bool isViewRule(char *relname);
 static void setMaxOid(Archive *fout);
 
 static void AddAcl(char *aclbuf, const char *keyword);
@@ -175,6 +182,7 @@ help(const char *progname)
 #ifdef HAVE_GETOPT_LONG
    puts(
        "  -a, --data-only          dump out only the data, not the schema\n"
+       "  -b, --blobs              dump out blob data\n"
        "  -c, --clean              clean (drop) schema prior to create\n"
        "  -C, --create             output commands to create database\n"
        "  -d, --inserts            dump data as INSERT, rather than COPY, commands\n"
@@ -200,6 +208,7 @@ help(const char *progname)
 #else
    puts(
        "  -a                       dump out only the data, no schema\n"
+       "  -b                       dump out blob data\n"
        "  -c                       clean (drop) schema prior to create\n"
        "  -C                       output commands to create database\n"
        "  -d                       dump data as INSERT, rather than COPY, commands\n"
@@ -246,48 +255,6 @@ exit_nicely(PGconn *conn)
 }
 
 
-/*
- * isViewRule
- *             Determine if the relation is a VIEW
- *
- */
-static bool
-isViewRule(char *relname)
-{
-   PGresult   *res;
-   int         ntups;
-   char       rulename[NAMEDATALEN + 5];
-   PQExpBuffer query = createPQExpBuffer();
-
-   appendPQExpBuffer(query, "select relname from pg_class, pg_rewrite ");
-   appendPQExpBuffer(query, "where pg_class.oid = ev_class ");
-   appendPQExpBuffer(query, "and pg_rewrite.ev_type = '1' ");
-   snprintf(rulename,NAMEDATALEN + 5,"_RET%s",relname);
-#ifdef MULTIBYTE
-   int len;
-   len = pg_mbcliplen(rulename,strlen(rulename),NAMEDATALEN-1);
-   rulename[len] = '\0';
-#else
-   rulename[NAMEDATALEN-1] = '\0';
-#endif
-
-   appendPQExpBuffer(query, "and rulename = '%s'", rulename);
-
-   res = PQexec(g_conn, query->data);
-   if (!res ||
-       PQresultStatus(res) != PGRES_TUPLES_OK)
-   {
-       fprintf(stderr, "isViewRule(): SELECT failed.  Explanation from backend: '%s'.\n", 
-                   PQerrorMessage(g_conn));
-       exit_nicely(g_conn);
-   }
-
-   ntups = PQntuples(res);
-
-   PQclear(res);
-   return ntups > 0 ? TRUE : FALSE;
-}
-
 #define COPYBUFSIZ     8192
 
 /*
@@ -602,7 +569,7 @@ dumpClasses(const TableInfo *tblinfo, const int numTables, Archive *fout,
        const char *classname = tblinfo[i].relname;
 
        /* Skip VIEW relations */
-       if (isViewRule(tblinfo[i].relname))
+       if (tblinfo[i].viewdef != NULL)
            continue;
 
        if (tblinfo[i].sequence)/* already dumped */
@@ -724,9 +691,9 @@ main(int argc, char **argv)
    }
 
 #ifdef HAVE_GETOPT_LONG
-   while ((c = getopt_long(argc, argv, "acCdDf:F:h:inNoOp:sS:t:uvxzZ:V?", long_options, &optindex)) != -1)
+   while ((c = getopt_long(argc, argv, "abcCdDf:F:h:inNoOp:sS:t:uvxzZ:V?", long_options, &optindex)) != -1)
 #else
-   while ((c = getopt(argc, argv, "acCdDf:F:h:inNoOp:sS:t:uvxzZ:V?-")) != -1)
+   while ((c = getopt(argc, argv, "abcCdDf:F:h:inNoOp:sS:t:uvxzZ:V?-")) != -1)
 #endif
 
    {
@@ -1097,12 +1064,13 @@ dumpBlobs(Archive *AH, char* junkOid, void *junkVal)
        fprintf(stderr, "%s saving BLOBs\n", g_comment_start);
 
    /* Cursor to get all BLOB tables */
-    appendPQExpBuffer(oidQry, "Declare blobOid Cursor for SELECT oid from pg_class where relkind = 'l'");
+    appendPQExpBuffer(oidQry, "Declare blobOid Cursor for SELECT oid from pg_class where relkind = '%c'", RELKIND_LOBJECT);
 
    res = PQexec(g_conn, oidQry->data);
    if (!res || PQresultStatus(res) != PGRES_COMMAND_OK)
    {
-       fprintf(stderr, "dumpBlobs(): Declare Cursor failed.  Explanation from backend: '%s'.\n", PQerrorMessage(g_conn));
+       fprintf(stderr, "dumpBlobs(): Declare Cursor failed.  Explanation from backend: '%s'.\n", 
+                                       PQerrorMessage(g_conn));
        exit_nicely(g_conn);
    }
 
@@ -1190,6 +1158,7 @@ getTypes(int *numTypes)
    int         i_typrelid;
    int         i_typbyval;
    int         i_usename;
+   int         i_typedefn;
 
    /* find all base types */
 
@@ -1204,7 +1173,9 @@ getTypes(int *numTypes)
 
    appendPQExpBuffer(query, "SELECT pg_type.oid, typowner, typname, typlen, typprtlen, "
          "typinput, typoutput, typreceive, typsend, typelem, typdelim, "
-        "typdefault, typrelid, typbyval, usename from pg_type, pg_user "
+        "typdefault, typrelid, typbyval, usename, "
+        "format_type(pg_type.oid, NULL) as typedefn "
+        "from pg_type, pg_user "
                      "where typowner = usesysid");
 
    res = PQexec(g_conn, query->data);
@@ -1234,6 +1205,7 @@ getTypes(int *numTypes)
    i_typrelid = PQfnumber(res, "typrelid");
    i_typbyval = PQfnumber(res, "typbyval");
    i_usename = PQfnumber(res, "usename");
+   i_typedefn = PQfnumber(res, "typedefn");
 
    for (i = 0; i < ntups; i++)
    {
@@ -1251,6 +1223,7 @@ getTypes(int *numTypes)
        tinfo[i].typdefault = strdup(PQgetvalue(res, i, i_typdefault));
        tinfo[i].typrelid = strdup(PQgetvalue(res, i, i_typrelid));
        tinfo[i].usename = strdup(PQgetvalue(res, i, i_usename));
+       tinfo[i].typedefn = strdup(PQgetvalue(res, i, i_typedefn));
 
        if (strcmp(PQgetvalue(res, i, i_typbyval), "f") == 0)
            tinfo[i].passedbyvalue = 0;
@@ -1492,6 +1465,8 @@ clearTableInfo(TableInfo *tblinfo, int numTables)
            free((int *) tblinfo[i].inhAttrs);
        if (tblinfo[i].attnames)
            free(tblinfo[i].attnames);
+       if (tblinfo[i].atttypedefns)
+           free(tblinfo[i].atttypedefns);
        if (tblinfo[i].typnames)
            free(tblinfo[i].typnames);
        if (tblinfo[i].notnull)
@@ -1825,6 +1800,12 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs)
    int         i_relchecks;
    int         i_reltriggers;
    int         i_relhasindex;
+   int         i_viewdef;
+
+   char        relkindview[2];
+
+   relkindview[0] = RELKIND_VIEW;
+   relkindview[1] = '\0';
 
    /*
     * find all the user-defined tables (no indices and no catalogs),
@@ -1832,17 +1813,18 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs)
     * tables before the child tables when traversing the tblinfo*
     *
     * we ignore tables that are not type 'r' (ordinary relation) or 'S'
-    * (sequence) --- in particular, Large Object relations (type 'l') are
-    * ignored.
+    * (sequence) or 'v' (view) --- in particular, Large Object 
+     * relations (type 'l') are ignored.
     */
 
    appendPQExpBuffer(query,
               "SELECT pg_class.oid, relname, relkind, relacl, usename, "
-                     "relchecks, reltriggers, relhasindex "
+                     "relchecks, reltriggers, relhasindex, pg_get_viewdef(relname) as viewdef "
                      "from pg_class, pg_user "
-                     "where relowner = usesysid and "
-               "(relkind = 'r' or relkind = 'S') and relname !~ '^pg_' "
-                     "order by oid");
+                     "where relowner = usesysid and relname !~ '^pg_' "
+                     "and relkind in ('%c', '%c', '%c') "
+                     "order by oid",
+               RELKIND_RELATION, RELKIND_SEQUENCE, RELKIND_VIEW);
 
    res = PQexec(g_conn, query->data);
    if (!res ||
@@ -1867,6 +1849,7 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs)
    i_relchecks = PQfnumber(res, "relchecks");
    i_reltriggers = PQfnumber(res, "reltriggers");
    i_relhasindex = PQfnumber(res, "relhasindex");
+   i_viewdef = PQfnumber(res, "viewdef");
 
    for (i = 0; i < ntups; i++)
    {
@@ -1877,6 +1860,11 @@ getTables(int *numTables, FuncInfo *finfo, int numFuncs)
        tblinfo[i].usename = strdup(PQgetvalue(res, i, i_usename));
        tblinfo[i].ncheck = atoi(PQgetvalue(res, i, i_relchecks));
        tblinfo[i].ntrig = atoi(PQgetvalue(res, i, i_reltriggers));
+       if (strcmp(PQgetvalue(res, i, i_relkind), relkindview) == 0) {
+           tblinfo[i].viewdef = strdup(PQgetvalue(res, i, i_viewdef));
+       } else {
+           tblinfo[i].viewdef = NULL;
+       }
 
        /*
         * Exclude inherited CHECKs from CHECK constraints total. If a
@@ -2361,6 +2349,7 @@ getTableAttrs(TableInfo *tblinfo, int numTables)
    int         i_attnotnull;
    int         i_atthasdef;
    int         i_attoid;
+   int         i_atttypedefn;
    PGresult   *res;
    int         ntups;
 
@@ -2385,7 +2374,7 @@ getTableAttrs(TableInfo *tblinfo, int numTables)
 
        resetPQExpBuffer(q);
        appendPQExpBuffer(q, "SELECT a.oid as attoid, a.attnum, a.attname, t.typname, a.atttypmod, "
-                         "a.attnotnull, a.atthasdef "
+                         "a.attnotnull, a.atthasdef, format_type(a.atttypid, a.atttypmod) as atttypedefn "
                          "from pg_attribute a, pg_type t "
                   "where a.attrelid = '%s'::oid and a.atttypid = t.oid "
                          "and a.attnum > 0 order by attnum",
@@ -2407,10 +2396,12 @@ getTableAttrs(TableInfo *tblinfo, int numTables)
        i_atttypmod = PQfnumber(res, "atttypmod");
        i_attnotnull = PQfnumber(res, "attnotnull");
        i_atthasdef = PQfnumber(res, "atthasdef");
+       i_atttypedefn = PQfnumber(res, "atttypedefn");
 
        tblinfo[i].numatts = ntups;
        tblinfo[i].attoids = (char **) malloc(ntups * sizeof(char *));
        tblinfo[i].attnames = (char **) malloc(ntups * sizeof(char *));
+       tblinfo[i].atttypedefns = (char **) malloc(ntups * sizeof(char *));
        tblinfo[i].typnames = (char **) malloc(ntups * sizeof(char *));
        tblinfo[i].atttypmod = (int *) malloc(ntups * sizeof(int));
        tblinfo[i].inhAttrs = (int *) malloc(ntups * sizeof(int));
@@ -2422,6 +2413,7 @@ getTableAttrs(TableInfo *tblinfo, int numTables)
        {
            tblinfo[i].attoids[j] = strdup(PQgetvalue(res, j, i_attoid));
            tblinfo[i].attnames[j] = strdup(PQgetvalue(res, j, i_attname));
+           tblinfo[i].atttypedefns[j] = strdup(PQgetvalue(res, j, i_atttypedefn));
            tblinfo[i].typnames[j] = strdup(PQgetvalue(res, j, i_typname));
            tblinfo[i].atttypmod[j] = atoi(PQgetvalue(res, j, i_atttypmod));
            tblinfo[i].inhAttrs[j] = 0; /* this flag is set in
@@ -2503,8 +2495,8 @@ getIndices(int *numIndices)
                    "from pg_index i, pg_class t1, pg_class t2, pg_am a "
                   "WHERE t1.oid = i.indexrelid and t2.oid = i.indrelid "
                      "and t1.relam = a.oid and i.indexrelid > '%u'::oid "
-                     "and t2.relname !~ '^pg_' and t2.relkind != 'l' and not i.indisprimary",
-                     g_last_builtin_oid);
+                     "and t2.relname !~ '^pg_' and t2.relkind != '%c' and not i.indisprimary",
+                     g_last_builtin_oid, RELKIND_LOBJECT);
 
    res = PQexec(g_conn, query->data);
    if (!res ||
@@ -2720,7 +2712,7 @@ dumpTypes(Archive *fout, FuncInfo *finfo, int numFuncs,
        {
            char       *elemType;
 
-           elemType = findTypeByOid(tinfo, numTypes, tinfo[i].typelem);
+           elemType = findTypeByOid(tinfo, numTypes, tinfo[i].typelem, zeroAsOpaque);
 
            appendPQExpBuffer(q, ", element = %s, delimiter = '%s'",
                              elemType, tinfo[i].typdelim);
@@ -2922,13 +2914,13 @@ dumpOneFunc(Archive *fout, FuncInfo *finfo, int i,
    {
        char            *typname;
 
-       typname = findTypeByOid(tinfo, numTypes, finfo[i].argtypes[j]);
-       appendPQExpBuffer(fn, "%s%s",
-                           (j > 0) ? "," : "",
-                           fmtId(typname, false));
+       typname = findTypeByOid(tinfo, numTypes, finfo[i].argtypes[j], zeroAsOpaque);
+       appendPQExpBuffer(fn, "%s%s", 
+                           (j > 0) ? "," : "", 
+                           typname);
        appendPQExpBuffer(fnlist, "%s%s",
-                           (j > 0) ? "," : "",
-                           fmtId(typname, false));
+                           (j > 0) ? "," : "", 
+                           typname );
    }
    appendPQExpBuffer(fn, ")");
 
@@ -2939,7 +2931,7 @@ dumpOneFunc(Archive *fout, FuncInfo *finfo, int i,
    appendPQExpBuffer(q, "CREATE FUNCTION %s ", fn->data );
    appendPQExpBuffer(q, "RETURNS %s%s %s LANGUAGE '%s'",
                      (finfo[i].retset) ? " SETOF " : "",
-                     fmtId(findTypeByOid(tinfo, numTypes, finfo[i].prorettype), false),
+                     findTypeByOid(tinfo, numTypes, finfo[i].prorettype, zeroAsOpaque),
                      asPart->data, func_lang);
 
    if (finfo[i].iscachable) /* OR in new attrs here */
@@ -3014,13 +3006,13 @@ dumpOprs(Archive *fout, OprInfo *oprinfo, int numOperators,
            strcmp(oprinfo[i].oprkind, "b") == 0)
        {
            appendPQExpBuffer(leftarg, ",\n\tLEFTARG = %s ",
-                             fmtId(findTypeByOid(tinfo, numTypes, oprinfo[i].oprleft), false));
+                               findTypeByOid(tinfo, numTypes, oprinfo[i].oprleft, zeroAsOpaque) );
        }
        if (strcmp(oprinfo[i].oprkind, "l") == 0 ||
            strcmp(oprinfo[i].oprkind, "b") == 0)
        {
            appendPQExpBuffer(rightarg, ",\n\tRIGHTARG = %s ",
-                             fmtId(findTypeByOid(tinfo, numTypes, oprinfo[i].oprright), false));
+                             findTypeByOid(tinfo, numTypes, oprinfo[i].oprright, zeroAsOpaque) );
        }
        if (!(strcmp(oprinfo[i].oprcom, "0") == 0))
            appendPQExpBuffer(commutator, ",\n\tCOMMUTATOR = %s ",
@@ -3046,11 +3038,9 @@ dumpOprs(Archive *fout, OprInfo *oprinfo, int numOperators,
 
        resetPQExpBuffer(delq);
        appendPQExpBuffer(delq, "DROP OPERATOR %s (%s", oprinfo[i].oprname,
-               fmtId(findTypeByOid(tinfo, numTypes, oprinfo[i].oprleft),
-                     false));
+               findTypeByOid(tinfo, numTypes, oprinfo[i].oprleft, zeroAsOpaque) );
        appendPQExpBuffer(delq, ", %s);\n",
-               fmtId(findTypeByOid(tinfo, numTypes, oprinfo[i].oprright),
-                    false));
+                   findTypeByOid(tinfo, numTypes, oprinfo[i].oprright, zeroAsOpaque) );
 
        resetPQExpBuffer(q);
        appendPQExpBuffer(q,
@@ -3098,12 +3088,12 @@ dumpAggs(Archive *fout, AggInfo *agginfo, int numAggs,
 
        appendPQExpBuffer(details,
                          "BASETYPE = %s, ",
-                         fmtId(findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype), false));
+                         findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype, zeroAsAny + useBaseTypeName));
 
        appendPQExpBuffer(details,
                          "SFUNC = %s, STYPE = %s",
                          agginfo[i].aggtransfn,
-                         fmtId(findTypeByOid(tinfo, numTypes, agginfo[i].aggtranstype), false));
+                         findTypeByOid(tinfo, numTypes, agginfo[i].aggtranstype, zeroAsOpaque + useBaseTypeName));
 
        if (agginfo[i].agginitval)
            appendPQExpBuffer(details, ", INITCOND = '%s'",
@@ -3115,7 +3105,7 @@ dumpAggs(Archive *fout, AggInfo *agginfo, int numAggs,
 
        resetPQExpBuffer(aggSig);
        appendPQExpBuffer(aggSig, "%s %s", agginfo[i].aggname,
-                           fmtId(findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype), false));
+                           findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype, zeroAsOpaque + useBaseTypeName));
 
        resetPQExpBuffer(delq);
        appendPQExpBuffer(delq, "DROP AGGREGATE %s;\n", aggSig->data);
@@ -3132,7 +3122,7 @@ dumpAggs(Archive *fout, AggInfo *agginfo, int numAggs,
 
        resetPQExpBuffer(q);
        appendPQExpBuffer(q, "AGGREGATE %s %s", agginfo[i].aggname,
-                         fmtId(findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype), false));
+                         findTypeByOid(tinfo, numTypes, agginfo[i].aggbasetype, zeroAsOpaque + useBaseTypeName));
        dumpComment(fout, q->data, agginfo[i].oid);
 
    }
@@ -3254,8 +3244,8 @@ dumpACL(Archive *fout, TableInfo tbinfo)
        eqpos = strchr(tok, '=');
        if (!eqpos)
        {
-           fprintf(stderr, "Could not parse ACL list for '%s'...Exiting!\n",
-                   tbinfo.relname);
+           fprintf(stderr, "Could not parse ACL list ('%s') for '%s'...Exiting!\n",
+                   acls, tbinfo.relname);
            exit_nicely(g_conn);
        }
 
@@ -3324,9 +3314,7 @@ dumpTables(Archive *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;
+   char       *reltypename;
 
    /* First - dump SEQUENCEs */
    if (tablename)
@@ -3359,121 +3347,94 @@ dumpTables(Archive *fout, TableInfo *tblinfo, int numTables,
        if (!tablename || (!strcmp(tblinfo[i].relname, tablename)))
        {
 
-           /* Skip VIEW relations */
-           
-           /* if (isViewRule(tblinfo[i].relname)) continue; */
-           
+           resetPQExpBuffer(delq);
+           resetPQExpBuffer(q);
 
-           parentRels = tblinfo[i].parentRels;
-           numParents = tblinfo[i].numParents;
+           /* Use the view definition if there is one */
+           if (tblinfo[i].viewdef != NULL)
+           {
+               reltypename = "VIEW";
 
-           resetPQExpBuffer(delq);
-           appendPQExpBuffer(delq, "DROP TABLE %s;\n", fmtId(tblinfo[i].relname, force_quotes));
+               appendPQExpBuffer(delq, "DROP VIEW %s;\n", fmtId(tblinfo[i].relname, force_quotes));
+               appendPQExpBuffer(q, "CREATE VIEW %s as %s", fmtId(tblinfo[i].relname, force_quotes), tblinfo[i].viewdef);
 
-           resetPQExpBuffer(q);
-           appendPQExpBuffer(q, "CREATE TABLE %s (\n\t", fmtId(tblinfo[i].relname, force_quotes));
-           actual_atts = 0;
-           for (j = 0; j < tblinfo[i].numatts; j++)
+           }
+           else
            {
-               if (tblinfo[i].inhAttrs[j] == 0)
-               {
-                   if (actual_atts > 0)
-                       appendPQExpBuffer(q, ",\n\t");
-                   appendPQExpBuffer(q, "%s ",
-                           fmtId(tblinfo[i].attnames[j], force_quotes));
+               reltypename = "TABLE";
 
-                   /* Show lengths on bpchar and varchar */
-                   if (!strcmp(tblinfo[i].typnames[j], "bpchar"))
-                   {
-                       int         len = (tblinfo[i].atttypmod[j] - VARHDRSZ);
+               parentRels = tblinfo[i].parentRels;
+               numParents = tblinfo[i].numParents;
 
-                       appendPQExpBuffer(q, "character");
-                       if (len > 1)
-                           appendPQExpBuffer(q, "(%d)",
-                                    tblinfo[i].atttypmod[j] - VARHDRSZ);
-                   }
-                   else if (!strcmp(tblinfo[i].typnames[j], "varchar"))
-                   {
-                       appendPQExpBuffer(q, "character varying");
-                       if (tblinfo[i].atttypmod[j] != -1)
-                       {
-                           appendPQExpBuffer(q, "(%d)",
-                                    tblinfo[i].atttypmod[j] - VARHDRSZ);
-                       }
-                   }
-                   else if (!strcmp(tblinfo[i].typnames[j], "numeric"))
-                   {
-                       appendPQExpBuffer(q, "numeric");
-                       if (tblinfo[i].atttypmod[j] != -1)
-                       {
-                           tmp_typmod = tblinfo[i].atttypmod[j] - VARHDRSZ;
-                           precision = (tmp_typmod >> 16) & 0xffff;
-                           scale = tmp_typmod & 0xffff;
-                           appendPQExpBuffer(q, "(%d,%d)",
-                                             precision, scale);
-                       }
-                   }
+               appendPQExpBuffer(delq, "DROP TABLE %s;\n", fmtId(tblinfo[i].relname, force_quotes));
 
-                   /*
-                    * char is an internal single-byte data type; Let's
-                    * make sure we force it through with quotes. - thomas
-                    * 1998-12-13
-                    */
-                   else if (!strcmp(tblinfo[i].typnames[j], "char"))
-                   {
-                       appendPQExpBuffer(q, "%s",
-                                   fmtId(tblinfo[i].typnames[j], true));
-                   }
-                   else
+               appendPQExpBuffer(q, "CREATE TABLE %s (\n\t", fmtId(tblinfo[i].relname, force_quotes));
+               actual_atts = 0;
+               for (j = 0; j < tblinfo[i].numatts; j++)
+               {
+                   /* Is this one of the table's own attrs ? */
+                   if (tblinfo[i].inhAttrs[j] == 0)
                    {
-                       appendPQExpBuffer(q, "%s",
-                                  fmtId(tblinfo[i].typnames[j], false));
-                   }
-                   if (tblinfo[i].adef_expr[j] != NULL)
-                       appendPQExpBuffer(q, " DEFAULT %s",
+                       /* Format properly if not first attr */
+                       if (actual_atts > 0)
+                           appendPQExpBuffer(q, ",\n\t");
+
+                       /* Attr name & type */
+                       appendPQExpBuffer(q, "%s %s",
+                           fmtId(tblinfo[i].attnames[j], force_quotes),
+                           tblinfo[i].atttypedefns[j]);
+
+                       /* Default value */
+                       if (tblinfo[i].adef_expr[j] != NULL)
+                           appendPQExpBuffer(q, " DEFAULT %s",
                                          tblinfo[i].adef_expr[j]);
-                   if (tblinfo[i].notnull[j])
-                       appendPQExpBuffer(q, " NOT NULL");
-                   actual_atts++;
+
+                       /* Not Null constraint */
+                       if (tblinfo[i].notnull[j])
+                           appendPQExpBuffer(q, " NOT NULL");
+
+                       actual_atts++;
+                   }
                }
-           }
 
-           /* put the CONSTRAINTS inside the table def */
-           for (k = 0; k < tblinfo[i].ncheck; k++)
-           {
-               if (actual_atts + k > 0)
-                   appendPQExpBuffer(q, ",\n\t");
-               appendPQExpBuffer(q, "%s",
+               /* put the CONSTRAINTS inside the table def */
+               for (k = 0; k < tblinfo[i].ncheck; k++)
+               {
+                   if (actual_atts + k > 0)
+                       appendPQExpBuffer(q, ",\n\t");
+
+                   appendPQExpBuffer(q, "%s",
                                  tblinfo[i].check_expr[k]);
-           }
+               }
 
-           /* PRIMARY KEY */
-           if (tblinfo[i].primary_key)
-           {
-               if (actual_atts + tblinfo[i].ncheck > 0)
-                   appendPQExpBuffer(q, ",\n\t");
-               appendPQExpBuffer(q, "PRIMARY KEY (%s)", tblinfo[i].primary_key);
-           }
+               /* PRIMARY KEY */
+               if (tblinfo[i].primary_key)
+               {
+                   if (actual_atts + tblinfo[i].ncheck > 0)
+                       appendPQExpBuffer(q, ",\n\t");
+                   appendPQExpBuffer(q, "PRIMARY KEY (%s)", tblinfo[i].primary_key);
+               }
 
-           appendPQExpBuffer(q, "\n)");
+               appendPQExpBuffer(q, "\n)");
 
-           if (numParents > 0)
-           {
-               appendPQExpBuffer(q, "\ninherits (");
-               for (k = 0; k < numParents; k++)
+               if (numParents > 0)
                {
-                   appendPQExpBuffer(q, "%s%s",
+                   appendPQExpBuffer(q, "\ninherits (");
+                   for (k = 0; k < numParents; k++)
+                   {
+                       appendPQExpBuffer(q, "%s%s",
                                      (k > 0) ? ", " : "",
                                      fmtId(parentRels[k], force_quotes));
+                   }
+                   appendPQExpBuffer(q, ")");
                }
-               appendPQExpBuffer(q, ")");
-           }
 
-           appendPQExpBuffer(q, ";\n");
+               appendPQExpBuffer(q, ";\n");
+           }
 
            if (!dataOnly) {
-               ArchiveEntry(fout, tblinfo[i].oid, fmtId(tblinfo[i].relname, false),
-                               "TABLE", NULL, q->data, delq->data, "", tblinfo[i].usename,
+                   ArchiveEntry(fout, tblinfo[i].oid, fmtId(tblinfo[i].relname, false),
+                               reltypename, NULL, q->data, delq->data, "", tblinfo[i].usename,
                                NULL, NULL);
            }
 
@@ -3494,7 +3455,7 @@ dumpTables(Archive *fout, TableInfo *tblinfo, int numTables,
            /* Dump Table Comments */
 
            resetPQExpBuffer(q);
-           appendPQExpBuffer(q, "TABLE %s", fmtId(tblinfo[i].relname, force_quotes));
+           appendPQExpBuffer(q, "%s %s", reltypename, fmtId(tblinfo[i].relname, force_quotes));
            dumpComment(fout, q->data, tblinfo[i].oid);
 
        }
@@ -3756,20 +3717,20 @@ setMaxOid(Archive *fout)
    char        sql[1024];
    int     pos;
 
-   res = PQexec(g_conn, "CREATE TEMPORARY TABLE pg_dump_oid (dummy int4)");
+   res = PQexec(g_conn, "CREATE TEMPORARY TABLE pgdump_oid (dummy int4)");
    if (!res ||
        PQresultStatus(res) != PGRES_COMMAND_OK)
    {
-       fprintf(stderr, "Can not create pg_dump_oid table.  "
+       fprintf(stderr, "Can not create pgdump_oid table.  "
                    "Explanation from backend: '%s'.\n", PQerrorMessage(g_conn));
        exit_nicely(g_conn);
    }
    PQclear(res);
-   res = PQexec(g_conn, "INSERT INTO pg_dump_oid VALUES (0)");
+   res = PQexec(g_conn, "INSERT INTO pgdump_oid VALUES (0)");
    if (!res ||
        PQresultStatus(res) != PGRES_COMMAND_OK)
    {
-       fprintf(stderr, "Can not insert into pg_dump_oid table.  "
+       fprintf(stderr, "Can not insert into pgdump_oid table.  "
                    "Explanation from backend: '%s'.\n", PQerrorMessage(g_conn));
        exit_nicely(g_conn);
    }
@@ -3780,11 +3741,11 @@ setMaxOid(Archive *fout)
        exit_nicely(g_conn);
    }
    PQclear(res);
-   res = PQexec(g_conn, "DROP TABLE pg_dump_oid;");
+   res = PQexec(g_conn, "DROP TABLE pgdump_oid;");
    if (!res ||
        PQresultStatus(res) != PGRES_COMMAND_OK)
    {
-       fprintf(stderr, "Can not drop pg_dump_oid table.  "
+       fprintf(stderr, "Can not drop pgdump_oid table.  "
                            "Explanation from backend: '%s'.\n", PQerrorMessage(g_conn));
        exit_nicely(g_conn);
    }
@@ -3792,11 +3753,11 @@ setMaxOid(Archive *fout)
    if (g_verbose)
        fprintf(stderr, "%s maximum system oid is %u %s\n",
                g_comment_start, max_oid, g_comment_end);
-   pos = snprintf(sql, 1024, "CREATE TEMPORARY TABLE pg_dump_oid (dummy int4);\n");
-   pos = pos + snprintf(sql+pos, 1024-pos, "COPY pg_dump_oid WITH OIDS FROM stdin;\n");
+   pos = snprintf(sql, 1024, "CREATE TEMPORARY TABLE pgdump_oid (dummy int4);\n");
+   pos = pos + snprintf(sql+pos, 1024-pos, "COPY pgdump_oid WITH OIDS FROM stdin;\n");
    pos = pos + snprintf(sql+pos, 1024-pos, "%-d\t0\n", max_oid);
    pos = pos + snprintf(sql+pos, 1024-pos, "\\.\n");
-   pos = pos + snprintf(sql+pos, 1024-pos, "DROP TABLE pg_dump_oid;\n");
+   pos = pos + snprintf(sql+pos, 1024-pos, "DROP TABLE pgdump_oid;\n");
 
    ArchiveEntry(fout, "0", "Max OID", "", NULL, sql, "", "", "", NULL, NULL);
 }
@@ -4015,13 +3976,18 @@ dumpRules(Archive *fout, const char *tablename,
 
        /*
         * Get all rules defined for this table
+        * We include pg_rules in the cross since it filters out
+        * all view rules (pjw 15-Sep-2000).
         */
        resetPQExpBuffer(query);
-       appendPQExpBuffer(query, "SELECT pg_get_ruledef(pg_rewrite.rulename) AS definition,"
-                         "pg_get_userbyid(pg_class.relowner) AS viewowner, "
-                         "pg_rewrite.oid, pg_rewrite.rulename FROM pg_rewrite, pg_class "
+       appendPQExpBuffer(query, "SELECT definition,"
+                         "   pg_get_userbyid(pg_class.relowner) AS viewowner, "
+                         "   pg_rewrite.oid, pg_rewrite.rulename "
+                         "FROM pg_rewrite, pg_class, pg_rules "
                          "WHERE pg_class.relname = '%s' "
-                         "AND pg_rewrite.ev_class = pg_class.oid "
+                         "    AND pg_rewrite.ev_class = pg_class.oid "
+                         "    AND pg_rules.tablename = pg_class.relname "
+                         "    AND pg_rules.rulename = pg_rewrite.rulename "
                          "ORDER BY pg_rewrite.oid",
                          tblinfo[t].relname);
        res = PQexec(g_conn, query->data);
index b687e42d6a83b189d37b7ecb7695153e18c5e02a..728905cdcbefe2700d1a2084a716ba00a3639576 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: pg_dump.h,v 1.51 2000/08/07 12:32:54 pjw Exp $
+ * $Id: pg_dump.h,v 1.52 2000/09/15 04:35:16 pjw Exp $
  *
  * Modifications - 6/12/96 - [email protected] - version 1.13.dhb.2
  *
  * Modifications - 6/1/97 - [email protected]
  * - Added extern's for the functions that clear allocated memory
  *  in pg_dump.c
+ *
+ * Modifications - 14-Sep-2000 - [email protected]
+ * -   Added typedefn fields to typeinfo and relinfo
+ *     -   Added enum for findTypeByOid to allow special handling of
+ *     '0' OID.
+ *
  *-------------------------------------------------------------------------
  */
 
@@ -45,6 +51,7 @@ typedef struct _typeInfo
    char       *typdefault;
    char       *typrelid;
    char       *usename;
+   char       *typedefn;
    int         passedbyvalue;
    int         isArray;
 } TypeInfo;
@@ -80,6 +87,7 @@ typedef struct _tableInfo
    char       *oid;
    char       *relname;
    char       *relacl;
+   char       *viewdef;
    bool        sequence;
    int         numatts;        /* number of attributes */
    int        *inhAttrs;       /* an array of flags, one for each
@@ -87,6 +95,7 @@ typedef struct _tableInfo
                                 * attribute is an inherited attribute */
    char      **attnames;       /* the attribute names */
    char      **attoids;        /* oids of the various attributes */
+   char      **atttypedefns;   /* formatted column type definitions */
    char      **typnames;       /* fill out attributes */
    bool       *notnull;        /* Not null constraints of an attribute */
    char      **adef_expr;      /* DEFAULT expressions */
@@ -197,7 +206,13 @@ extern void dumpSchemaIdx(Archive *fout,
              TableInfo *tblinfo,
              int numTables);
 
-extern char *findTypeByOid(TypeInfo *tinfo, int numTypes, const char *oid);
+typedef enum _OidOptions {
+   zeroAsOpaque = 1,
+   zeroAsAny = 2,
+    useBaseTypeName = 1024
+} OidOptions;
+
+extern char *findTypeByOid(TypeInfo *tinfo, int numTypes, const char *oid, OidOptions opts);
 extern char *findOprByOid(OprInfo *oprinfo, int numOprs, const char *oid);
 extern int findFuncByName(FuncInfo *finfo, int numFuncs, const char *name);
 extern int findTableByName(TableInfo *tbinfo, int numTables, const char *relname);