Don't try to assign smart names to constraints

This part of my previous commit seems to have broken pg_upgrade on
crake, at least from 9.2.  I'll see if there's a better fix, but in the
meantime this should suffice to keep the buildfarm green.

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 13a6bce5119d0f932ffac78428a4523faf12d2c3..6d2f3fdef388c6581ebb7d695fed52d7d49c9578 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -9097,20 +9097,10 @@ getTableAttrs(Archive *fout, TableInfo *tblinfo, int numTables)
            else if (use_throwaway_notnull)
            {
                /*
-                * Decide on a name for this constraint.  If it is not an
-                * inherited constraint, give it a throwaway name to avoid any
-                * possible conflicts, since we're going to drop it soon
-                * anyway.  If it is inherited then try harder, because it may
-                * (but not necessarily) persist after the restore.
+                * Give this constraint a throwaway name.
                 */
-               if (tbinfo->notnull_inh[j])
-                   /* XXX maybe try harder if the name is overlength */
-                   tbinfo->notnull_constrs[j] =
-                       psprintf("%s_%s_not_null",
-                                tbinfo->dobj.name, tbinfo->attnames[j]);
-               else
-                   tbinfo->notnull_constrs[j] =
-                       psprintf("pgdump_throwaway_notnull_%d", notnullcount++);
+               tbinfo->notnull_constrs[j] =
+                   psprintf("pgdump_throwaway_notnull_%d", notnullcount++);
                tbinfo->notnull_throwaway[j] = true;
                tbinfo->notnull_inh[j] = false;
            }