On 2024-09-09 16:56, Michael Harris wrote:
Thanks for updating the patch.
Here is a minor comment.
@@ -944,20 +948,32 @@ expand_vacuum_rel(VacuumRelation *vrel,
MemoryContext vac_context,
MemoryContextSwitchTo(oldcontext);
}
..
+ * Unless the user has specified ONLY, make relation list
entries for
+ * its partitions and/or descendant tables.
Regarding the addition of partition descendant tables, should we also
update the below comment on expand_vacuum_rel? Currently it refers only
partitions:
| * Given a VacuumRelation, fill in the table OID if it wasn't
specified,
| * and optionally add VacuumRelations for partitions of the table.
Other than this and the following, it looks good to me.
On Mon, Sep 9, 2024 at 10:27 AM David Rowley <dgrowle...@gmail.com>
wrote:
Aside from those, that just leaves me with the behavioural change. I
noted Tom was ok with the change in behaviour for ANALYZE (mentioned
in [1]). Tom, wondering if you feel the same for VACUUM too? If we're
doing this, I think we'd need to be quite clear about it on the
release notes.
--
Regards,
--
Atsushi Torikoshi
NTT DATA Group Corporation
