Re: [COMMITTERS] pgsql: Allow multiple tables to be specified in one VACUUM or ANALYZE c
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Michael Paquier <michael.paquier@gmail.com>
Cc: PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2017-10-05T14:52:27Z
Lists: pgsql-hackers
Michael Paquier <michael.paquier@gmail.com> writes: > Tom, it seems to me that in the portions you have editorialized, you > have forgotten to update two comments still mentioning get_rel_oids() > in vacuum.c and analyze.c. Those should now refer to > expand_vacuum_rel() instead. Please see the attached. Oh, good point. I don't think that just s/get_rel_oids/expand_vacuum_rel/ will do though, as it leaves out the interaction with get_all_vacuum_rels. I decided that the point was of merely historical interest anyway, and just removed the reference to the other routine. The partitioned-table patches seem to have been a bit sloppy with maintaining these comments too, so I ended up doing more than that... regards, tom lane
Commits
-
Improve comments in vacuum_rel() and analyze_rel().
- 4d85c2900b11 11.0 landed
-
Allow multiple tables to be specified in one VACUUM or ANALYZE command.
- 11d8d72c27a6 11.0 cited