In dumpTable, re-instate the skipping logic

Pretty sure I removed this based on some incorrect thinking that it was
no longer possible to reach this point for a table which will not be
dumped, but that's clearly wrong.

Pointed out on IRC by Erik Rijkers.

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 33cd6651d124af78ad257e2d4c4619cad734b9de..6c2167616dd802a7c25d09380c338660d36ce931 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -14846,6 +14846,13 @@ dumpTable(Archive *fout, TableInfo *tbinfo)
        DumpOptions *dopt = fout->dopt;
        char       *namecopy;
 
+       /*
+        * noop if we are not dumping anything about this table, or if we are
+        * doing a data-only dump
+        */
+       if (!tbinfo->dobj.dump || dopt->dataOnly)
+               return;
+
        if (tbinfo->relkind == RELKIND_SEQUENCE)
                dumpSequence(fout, tbinfo);
        else