database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-04-27 22:56:53 +03:00
Code

Clean up useless "skipping" messages for VACUUM/ANALYZE.

Browse Source 
When VACUUM/ANALYZE are run on an entire database, it warns of
skipping relations for which the user doesn't have sufficient
privileges. That only makes sense for tables, so skip such messages
for indexes, etc.

Reviewed-by: Nathan Bossart
Discussion: https://postgr.es/m/c0a85c2e83158560314b576b6241c8ed0aea1745.camel%40j-davis.com
This commit is contained in:
Jeff Davis 2023-01-13 14:42:03 -08:00
parent c44f6334ca
commit d46a9792a8
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/backend/commands/vacuum.c
View File

@ -906,10 +906,6 @@ get_all_vacuum_rels(int options)
MemoryContext oldcontext;
Oid relid = classForm->oid;
/* check permissions of relation */
if (!vacuum_is_permitted_for_relation(relid, classForm, options))
continue;
/*
* We include partitioned tables here; depending on which operation is
* to be performed, caller will decide whether to process or ignore
@ -920,6 +916,10 @@ get_all_vacuum_rels(int options)
classForm->relkind != RELKIND_PARTITIONED_TABLE)
continue;
/* check permissions of relation */
if (!vacuum_is_permitted_for_relation(relid, classForm, options))
continue;
/*
* Build VacuumRelation(s) specifying the table OIDs to be processed.
* We omit a RangeVar since it wouldn't be appropriate to complain