Fix omission of column-level privileges in selective pg_restore.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 2 Oct 2023 17:27:51 +0000 (13:27 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 2 Oct 2023 17:27:51 +0000 (13:27 -0400)
commit4f16152d9711a59e1b2ad8a4bc26d4455da709e6
treef54b181d36f336f1b163f7dae5b8c4eb4dafe924
parent7a310cae0861cfd3a76ce4ace02ed72641039e15
Fix omission of column-level privileges in selective pg_restore.

In a selective restore, ACLs for a table should be dumped if the
table is selected to be dumped.  However, if the table has both
table-level and column-level ACLs, only the table-level ACL was
restored.  This happened because _tocEntryRequired assumed that
an ACL could have only one dependency (the one on its table),
and punted if there was more than one.  But since commit ea9125304,
column-level ACLs also depend on the table-level ACL if any, to
ensure correct ordering in parallel restores.  To fix, adjust the
logic in _tocEntryRequired to ignore dependencies on ACLs.

I extended a test case in 002_pg_dump.pl so that it purports to
test for this; but in fact the test passes even without the fix.
That's because this bug only manifests during a selective restore,
while the scenarios 002_pg_dump.pl tests include only selective dumps.
Perhaps somebody would like to extend the script so that it can test
scenarios including selective restore, but I'm not touching that.

Euler Taveira and Tom Lane, per report from Kong Man.
Back-patch to all supported branches.

Discussion: http://postgr.es/m/DM4PR11MB73976902DBBA10B1D652F9498B06A@DM4PR11MB7397.namprd11.prod.outlook.com
src/bin/pg_dump/pg_backup_archiver.c
src/bin/pg_dump/t/002_pg_dump.pl