pgsql: Minor cleanup/optimization in pg_dump. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Minor cleanup/optimization in pg_dump.
Date
Msg-id E1n317N-0001a2-Kr@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Minor cleanup/optimization in pg_dump.

In the wake of commits 05649b88c and 5209c0ba0, findComments() and
findSecLabels() no longer use their "Archive *fout" arguments,
so get rid of those.

While doing that, I noticed that there's no very good reason why
dumpCompositeTypeColComments() should be doing its own query to fetch
the column names of the composite type, when the calling function has
just fetched the same data.  Tweak it to use that query result.  This
probably doesn't save a lot for most people, because since 5209c0ba0
we won't get into this code at all unless the composite type has at
least one comment.  Nonetheless, it's a wasted query.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c7cf73eb7b9e7911748ebe117a7219f21e504121

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 84 ++++++++++++++++-------------------------------
1 file changed, 29 insertions(+), 55 deletions(-)


pgsql-committers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: pgsql: Revert b2a459edf "Fix GRANTED BY support in REVOKE ROLE statemen
Next
From: Alvaro Herrera
Date:
Subject: pgsql: Small cleanups related to PUBLICATION framework code