Обсуждение: Re: add missing PQfinish() calls to vacuumdb

> On 4 Feb 2025, at 17:30, Nathan Bossart <nathandbossart@gmail.com> wrote:
> 
> I noticed that vacuum_one_database() doesn't call PQfinish() before
> pg_fatal() in a few of the server version checks.  I seem to have
> unintentionally established this precedent in commit 00d1e88.  Michael
> claimed to have fixed it before committing [0], but that seems to have been
> missed, too.  I don't think this is a huge problem, but it does seem nicer
> to properly close the connection before exiting.  If there are no
> objections, I plan to commit and back-patch the attached patch shortly.

No objections, I too prefer to do the right thing here. +1 on backpatching.

--
Daniel Gustafsson

On Tue, Feb 04, 2025 at 05:51:28PM +0100, Daniel Gustafsson wrote:
> No objections, I too prefer to do the right thing here. +1 on backpatching.

Committed.  Thanks for looking.

-- 
nathan