On Tue, Aug 11, 2026 at 09:53:53AM -0500, Sami Imseih wrote: > I looked these patches while looking at potential cases for injection point > conditional strings [0], which could allow for better autovacuum testing, > particularly for the case being discussed here.
Thanks for reviewing. > 1/ Instead of setting toast_vacuum_params.* inside each branch, it reads > cleaner to set toast_vacuum_params.* after params.* is done, and this > way you only set toast_vacuum_params.*once. For example, in > index_cleanup: Done in v8 [0]. > 2/ A couple of comment tweaks in merge_autovac_opts() I don't think these changes apply to v8, except for perhaps the "NB" one, which I expanded a bit. [0] https://postgr.es/m/anuXd35G8WLTUaBR%40nathan -- nathan
