pgsql: Reorder code in pg_dump to dump comments etc in a uniform order.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Reorder code in pg_dump to dump comments etc in a uniform order.
Date: 2018-01-22 17:37:15
Message-ID: E1edg1z-00011y-73@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Reorder code in pg_dump to dump comments etc in a uniform order.

Most of the code in pg_dump dumps an object's comment, security label,
and ACL auxiliary TOC entries, in that order, immediately after the
object's main TOC entry, and at least dumpComment's API spec says this
isn't optional. dumpDatabase was significantly violating that when
in binary-upgrade mode, by inserting totally unrelated stuff between.
Also, dumpForeignDataWrapper and dumpForeignServer were being randomly
inconsistent. Reorder code so everybody does it the same.

This may be future-proofing us against some code growing a requirement for
such auxiliary entries to be adjacent to their main entry. But for now
it's just neatnik-ism, so I see no need for back-patch.

Discussion: https://postgr.es/m/21714.1516553459@sss.pgh.pa.us

Branch
------
master

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

Modified Files
--------------
src/bin/pg_dump/pg_dump.c | 150 +++++++++++++++++++++++-----------------------
1 file changed, 75 insertions(+), 75 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2018-01-22 19:09:20 pgsql: Move handling of database properties from pg_dumpall into pg_dum
Previous Message Peter Eisentraut 2018-01-22 17:11:30 pgsql: PL/Python: Fix tests for older Python versions