Hi Quan, wenhui,

Thanks for the review and feedback.

On Thu, Jun 5, 2026 at 09:25:52 AM, Quan Zongliang wrote:
> In the vacuum_all_databases function. If the condition 'if
> (dbsToExclude)' is used. All 'exclude-database' processing (including
> variable definitions) can be placed in a separate code block.

Good suggestion. Attached is v2 incorporating this change, the
exclude-database logic (including the `cell` and `first` variable
declarations) is now scoped inside an `if (dbsToExclude)` block.
This keeps the variables out of the function's top-level scope when
the feature isn't being used.


The patch is also tracked in CommitFest PG20-1:
https://commitfest.postgresql.org/patch/6833/

-- 
Mohamed Ali
AWS RDS

Attachment: v2-0001-vacuumdb-Add-exclude-database-option-to-skip-data.patch
Description: Binary data

Reply via email to