> One minor comment is: > > toast_vacuum_params.options |= VACOPT_PROCESS_MAIN; > + toast_vacuum_params.options &= ~VACOPT_ANALYZE; > toast_vacuum_params.toast_parent = relid; > > I wonder if this change is still necessary now that the filtering for TOAST > tables has been added to pgstat_report_skipped_vacuum_analyze(). > > If we keep this change, should we also add a comment explaining > why it is needed?
Right. I removed it. -- Sami
v17-0001-Track-skipped-vacuum-and-analyze-activity-per-re.patch
Description: Binary data
v17-0002-Add-injection-point-test-for-vacuum-skip_locked-.patch
Description: Binary data
