forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit0c9d844
committed
Rethink pg_dump's handling of object ACLs.
Throw away most of the existing logic for this, as it was veryinefficient thanks to expensive sub-selects executed to collectACL data that we very possibly would have no interest in dumping.Reduce the ACL handling in the initial per-object-type queriesto be just collection of the catalog ACL fields, as it wasoriginally. Fetch pg_init_privs data separately in a singlescan of that catalog, and do the merging calculations on theclient side. Remove the separate code path used for pre-9.6source servers; there is no good reason to treat them differentlyfrom newer servers that happen to have empty pg_init_privs.Discussion:https://postgr.es/m/2273648.1634764485@sss.pgh.pa.usDiscussion:https://postgr.es/m/7d7eb6128f40401d81b3b7a898b6b4de@W2012-02.nidsa.loc1 parent5209c0b commit0c9d844
File tree
7 files changed
+784
-1137
lines changed- src
- bin/pg_dump
- fe_utils
- include/fe_utils
7 files changed
+784
-1137
lines changed0 commit comments
Comments
(0)